Reverse Router.cy.js & Fix unwritable config after Cypress tests

Author: mahmoudmagdy1-1

.devcontainer/post-create.sh

@@ -124,6 +124,7 @@ service apache2 restart
 echo "--> Applying final group ownership and permissions..."
 chgrp -R www-data $JOOMLA_ROOT
 chmod -R g+rws $JOOMLA_ROOT
+chown www-data:www-data configuration.php
 
 echo "✅ Environment finalized."
 

tests/System/integration/site/components/com_contact/Router.cy.js

@@ -44,22 +44,16 @@ describe('Test in frontend that the contact site router', () => {
         expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_contact&view=contact&id=${contact.id}-${contact.alias}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_contact&view=contact&id=${contact.id}&catid=${contact.catid}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_contact&view=contact&id=${contact.id}-${contact.alias}&catid=${contact.catid}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
 
       cy.visit(url);
@@ -87,10 +81,8 @@ describe('Test in frontend that the contact site router', () => {
         expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_contact&view=contact&id=${contact.id}-${contact.alias}&catid=${contact.catid}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
 
       cy.visit(url.split('/').slice(0, -1).join('/'));

tests/System/integration/site/components/com_content/Router.cy.js

@@ -43,22 +43,16 @@ describe('Test in frontend that the content site router', () => {
         expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_content&view=article&id=${article.id}-${article.alias}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_content&view=article&id=${article.id}&catid=${article.catid}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_content&view=article&id=${article.id}-${article.alias}&catid=${article.catid}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
 
       cy.visit(url);
@@ -85,10 +79,8 @@ describe('Test in frontend that the content site router', () => {
         expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
       cy.request({ url: `/index.php?option=com_content&view=article&id=${article.id}-${article.alias}&catid=${article.catid}`, followRedirect: false }).then((response) => {
-        // @TODO: The following line is just a temporary fix. Remove it and uncomment the 2 lines after it when done.
-        expect(response.status).to.eq(200);
-        // expect(response.status).to.eq(301);
-        // expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
+        expect(response.status).to.eq(301);
+        expect(response.redirectedToUrl).to.match(new RegExp(`${url}$`));
       });
 
       cy.visit(url.split('/').slice(0, -1).join('/'));