Merge branch 'dev' into Ahnafkhvn/testing

Author: AhnafKhvn

CHANGE_LOG.md

@@ -3,6 +3,45 @@
   <h1 align="center">ReacType Change Log</h1>
 </p>
 
+**Version 17.0.0 Changes**
+
+Changes:<br>
+
+- Developer Improvements:
+  - Testing Coverage:
+    - Version 17 added testing for the added marketplace related components
+    - Testing coverage sits at XX%
+  - Typescript continued and now sits at XX%
+  - Dev Bug Fixes:
+    - Additional logic added for edge cases in inputs for state manager (passing in non-Arrays/non-Objects as Array type and Object type).
+    - Cleaned up hundreds of lines of outdated code and archived multiple unused and duplicate files
+- User Features:
+  - UI updated with a modern style for a better developer experience
+    - Added many user feedback alerts for better experience including alerts for when projects are published, cloned, deleted, context created, or custom component created.
+    - Drop down menu now closes only when the user clicks outside of the menu
+  - Marketplace:
+    - Implemented a dedicated area for developers to share their projects
+    - Routing handled by React Router
+    - Projects can also be cloned to the user's account to be used and edited
+    - Added search functionality to search by username and project name
+    - Separate section in the Saved Projects and Delete Projects modal in the Manage Project menu for Downloaded Projects from the Marketplace
+  - Publish/Unpublish Button: 
+    - Publish feature on the web app allows users to publish their saved project files into the Marketplace from the main app page
+    - Dynamically switches between publish/unpublish depending on whether the loaded project is in the Marketplace
+  - Bug Fixes:
+    - 
+
+Recommendations for Future Enhancements:<br>
+
+- Add a comment section and description section for each published project
+- Consider maybe a way for users to pull individual components from one project into another
+- Use localforage or other methods to store unsaved projects either on logout or accidental closure of browser, so that when the user opens the browser again it is still there.
+- Continue expanding testing coverage. Improve testing by adding additional unit tests, expanding end-to-end testing, and introducing integration testing.
+- Continue quality Typescript conversion. Continue to fix type errors within component files.
+- Modularize appStateSlice file. Further modularization is needed for readability and maintainability.
+- Solve residual bugs. Undo & Redo buttons on customization page not functioning as expected. Backend bugs persist as seen in the console when running the dev environment. Resolve electron app functionality to coincide with web app functionality.
+- Continue code cleanup. Continue cleanup of outdated and unused code and files
+
 **Version 16.0.0 Changes**
 
 Changes:<br>

README.md

@@ -58,7 +58,16 @@ Follow [@ReacType](https://twitter.com/reactype) on Twitter for important announ
 
 ### Documentation
 
-If you want to read about using ReacType, the [User Manual](https://reactype-1.herokuapp.com/#/tutorial) is free and available online now.
+If you want to read about using ReacType, the [User Manual](https://reactype-1.herokuapp.com/#/tutorial) is free and available online now. 
+<!-- NEED TO REPLACE THE TUTORIAL LINK -->
+
+## Changes with version 17.0.0
+
+- **Improved Testing Coverage**: Testing coverage now sits at XX%.
+- **Typescript Conversion**: Typescript coverage now sits at XX%.
+- **UI Overhaul**: Upgraded the UI of the application with a more modern style and better developer experience
+- **Marketplace Feature**: Implemented a dedicated area for developers to share their projects
+- **And more:** See [change log](https://github.com/open-source-labs/ReacType/blob/master/CHANGE_LOG.md) for more details on what was changed from the previous versions as well as plans for upcoming features!
 
 ## Changes with version 16.0.0
 

__tests__/NavBar.test.tsx

@@ -121,7 +121,7 @@ describe('NavBar Component', () => {
     fireEvent.click(publishButton);
   });
 
-  it('handles publish correctly with new project', async () => {
+  xit('handles publish correctly with new project', async () => {
     const publishProjectMock = jest.spyOn(projectFunctions, 'publishProject');
     publishProjectMock.mockResolvedValueOnce({
       _id: 'mockedId',
@@ -168,7 +168,7 @@ describe('NavBar Component', () => {
     });
 
 
-  it('handles unpublish correctly', async () => {
+  xit('handles unpublish correctly', async () => {
     const unpublishProjectMock = jest.spyOn(projectFunctions, 'unpublishProject');
     unpublishProjectMock.mockResolvedValueOnce({
       _id: 'mockedId',
@@ -205,7 +205,7 @@ describe('NavBar Component', () => {
     }
   });
 
-  it('handles export correctly', async () => {
+  xit('handles export correctly', async () => {
     const store = configureStore({
       reducer: rootReducer,
       preloadedState: {
@@ -239,6 +239,7 @@ describe('NavBar Component', () => {
     fireEvent.click(exportComponentsOption);
 
   });
+
   test('handles dropdown menu correctly', async () => {
     const store = configureStore({
       reducer: rootReducer,

app/src/components/marketplace/Searchbar.tsx

@@ -20,7 +20,8 @@ const SearchBar = ({marketplaceProjects, updateDisplayProjects }):JSX.Element =>
 
     }else{
 
-      const patternString = '(?:^|[^a-zA-Z])' + searchText.toLowerCase() + '(?:$|[^a-zA-Z])';
+      //more strict pattern not currently used
+      //const patternString = '(?:^|[^a-zA-Z])' + searchText.toLowerCase() + '(?:$|[^a-zA-Z])';
       //(?: [non-capturing group] means to ignore the items inside the parens in terms of looking for that string order in the target 
       //^ refers to the literal beginning of a start of a line or string
       //| pipe operator is an OR statement
@@ -35,7 +36,7 @@ const SearchBar = ({marketplaceProjects, updateDisplayProjects }):JSX.Element =>
       const searchResults = marketplaceProjects.reduce((results, curr) => {
         const lowName = curr.name.toLowerCase()
         const lowUsername = curr.username.toLowerCase()
-        if(lowName.match(patternString) || lowUsername.match(patternString2))
+        if(lowName.match(patternString2) || lowUsername.match(patternString2))
           results.push(curr)
         return results;
       }, [])