bluehost-wordpress-plugin
                                
                                
                                
                                    bluehost-wordpress-plugin copied to clipboard
                            
                            
                            
                        Clean up plugin code to pass phpcs checks
Proposed changes
Updates the codebase to pass code standards and linting from #851.
Branched off and targeted towards feature/code-standards.
34 failed and 1 flaky tests on run #5940 ↗︎
Details:
| Fix removed composer deps | |||
| Project: Bluehost Brand Plugin | Commit: e2c7ac882a | 
  ||
| Status: Failed | Duration: 19:20 💡 | ||
| Started: Dec 6, 2023 7:04 PM | Ended: Dec 6, 2023 7:24 PM | ||
   tests/cypress/integration/navigation.cy.js • 2 failed tests  
  | Test | Artifacts | |
|---|---|---|
| Navigation > Main nav links properly navigates | 
          
            Test Replay
           
            Screenshots
           
            Video
          
         | 
      |
| Navigation > Subnav links properly navigates | 
          
            Test Replay
           
            Screenshots
           
            Video
          
         | 
      |
   vendor/newfold-labs/wp-module-ctb/tests/cypress/integration/ctb.cy.js • 3 failed tests  
  
   vendor/newfold-labs/wp-module-global-ctb/tests/cypress/integration/global-ctb.cy.js • 3 failed tests  
  
   vendor/newfold-labs/wp-module-marketplace/tests/cypress/integration/marketplace.cy.js • 9 failed tests  
  
   vendor/newfold-labs/wp-module-patterns/tests/cypress/integration/wonder-blocks.cy.js • 3 failed tests  
  The first 5 failed specs are shown, see all 7 specs in Cypress Cloud.
   tests/cypress/integration/help.cy.js • 1 flaky test  
  | Test | Artifacts | |
|---|---|---|
| Help Page > Is Accessible | 
          
            Test Replay
           
            Screenshots
           
            Video
          
         | 
      |
Review all test suite changes for PR #852 ↗︎
Now that tests are passing on main, maybe we revisit this.
Even though merging main into these branches will be full of conflicts...