graphdb-workbench-tests 2.8.1-TR1 → 2.8.2-TR1

package/fixtures/locale-en.json

@@ -269,7 +269,7 @@
             "loading_rules": "Error during ACL rules",
             "updating_rules": "Error during ACL rules update",
             "duplicated_rules": "Every ACL rule should be unique.",
-            "role_length_too_short": "Too short"
+            "role_length_too_short": "Too short or not \"*\""
         },
         "defaults": {
             "asterisk": "* - Any RDF value",
@@ -1755,6 +1755,7 @@
     "delete.repo.warning.msg": "<p>Are you sure you want to delete the repository <strong>{{repositoryId}}</strong>?</p><p><span class=\"icon-2x icon-warning\" style=\"color: var(--primary-color-dark)\"></span>All data in the repository will be lost.</p>",
     "confirm.restart.repo": "Confirm restart",
     "confirm.restart.repo.warning.msg": "<p>Are you sure you want to restart the repository <strong>{{repositoryId}}</strong>?</p><p><span class=\"icon-2x icon-warning\" style=\"color: var(--primary-color-dark)\"></span>The repository will be shut down immediately and all running queries and updates will be cancelled.</p>",
+    "restart.repo.in.cluster.tooltip": "Restarting a repository is not supported in a cluster environment. To apply configuration changes, restart all cluster nodes.",
     "location.cannot.be.empty.error": "Location cannot be empty",
     "required.field": "This field is required",
     "created.repo.success.msg": "The repository {{repoId}} has been created.",

package/fixtures/repositories/get-remote-and-local-repositories.json

@@ -0,0 +1,34 @@
+{
+    "": [
+        {
+            "id": "test",
+            "title": "",
+            "uri": "http://localhost:8080/graphdb/repositories/test",
+            "externalUrl": "http://boyantonchev:9000/repositories/test",
+            "local": true,
+            "type": "graphdb",
+            "sesameType": "graphdb:SailRepository",
+            "location": "",
+            "readable": true,
+            "writable": true,
+            "unsupported": false,
+            "state": "RUNNING"
+        }
+    ],
+    "http://localhost:7201": [
+        {
+            "id": "movies",
+            "title": "",
+            "uri": "http://localhost:7202/repositories/movies",
+            "externalUrl": "http://localhost:7202/repositories/movies",
+            "local": false,
+            "type": "graphdb",
+            "sesameType": "graphdb:SailRepository",
+            "location": "http://localhost:7202",
+            "readable": true,
+            "writable": true,
+            "unsupported": false,
+            "state": "RUNNING"
+        }
+    ]
+}

package/integration/repository/repositories.spec.js

@@ -5,6 +5,8 @@ import {GlobalOperationsStatusesStub} from "../../stubs/global-operations-status
 import {ModalDialogSteps} from "../../steps/modal-dialog-steps";
 import {ImportUserDataSteps} from "../../steps/import/import-user-data-steps";
 import {ImportSettingsDialogSteps} from "../../steps/import/import-settings-dialog-steps";
+import {ClusterStubs} from "../../stubs/cluster/cluster-stubs";
+import {RepositoriesStubs} from "../../stubs/repositories/repositories-stubs";
 
 describe('Repositories', () => {
 
@@ -503,6 +505,43 @@ describe('Repositories', () => {
         ModalDialogSteps.verifyDialogBody('Changing the repository ID is a dangerous operation since it renames the repository folder and enforces repository shutdown.');
     });
 
+    it('should NOT allow restart of LOCAL repository from EDIT PAGE, if node is in cluster', () => {
+        // Given I create a repository
+        cy.createRepository({id: repositoryId});
+        // When I set the node in a cluster
+        GlobalOperationsStatusesStub.stubGlobalOperationsStatusesResponse(repositoryId);
+        // Then go to the local repository's edit page
+        RepositorySteps.visitEditPage(repositoryId);
+        // I expect the repository restart checkbox button to be disabled
+        RepositorySteps.getEditViewRestartButton().should('be.visible');
+        RepositorySteps.getEditViewRestartButton().should('be.disabled');
+    });
+
+    it('should NOT allow restart of LOCAL repositories from REPOSITORIES PAGE, if node is in cluster', () => {
+        // Given I create a repository
+        cy.createRepository({id: repositoryId});
+        // When I set the node in a cluster
+        GlobalOperationsStatusesStub.stubGlobalOperationsStatusesResponse(repositoryId);
+        ClusterStubs.stubClusterNodeStatus();
+        // Then go to the repositories page
+        RepositorySteps.visit();
+        // I expect the local repository's restart button to be disabled
+        RepositorySteps.getRepositoryRestartButton(repositoryId).should('be.visible');
+        RepositorySteps.getRepositoryRestartButton(repositoryId).should('be.disabled');
+    });
+
+    it('should ALLOW restart of REMOTE repositories from REPOSITORIES PAGE, if node is in cluster', () => {
+        // Given I have a remote location
+        RepositoriesStubs.stubRepositories(0, '/repositories/get-remote-and-local-repositories.json');
+        RepositoriesStubs.stubLocations();
+        // When I set the node in a cluster
+        ClusterStubs.stubClusterNodeStatus();
+        // Then go to the repositories page
+        RepositorySteps.visit();
+        // Then I expect the remote repository's restart button to be enabled
+        RepositorySteps.getRestartRemoteRepoButton(0).should('be.enabled');
+    });
+
     function interceptRulesetFileUpload() {
        cy.intercept(
             {

package/integration/setup/aclmanagement/create-rule.spec.js

@@ -211,6 +211,10 @@ describe('ACL Management: create rule', () => {
         AclManagementSteps.addRuleInBeginning();
         AclManagementSteps.selectPolicy(0, 'allow');
         AclManagementSteps.fillRole(0, 'A');
+        // And I expect the prefix warning to NOT appear
+        AclManagementSteps.getPrefixWarning(0).should('not.exist');
+        // And I expect the prefix warning icon to NOT appear
+        AclManagementSteps.getWarningIcon(0).should('not.exist');
         AclManagementSteps.selectOperation(0, 'write');
         AclManagementSteps.fillSubject(0, '<urn:Mary>');
         AclManagementSteps.fillPredicate(0, '*');
@@ -218,14 +222,39 @@ describe('ACL Management: create rule', () => {
         AclManagementSteps.fillContext(0, '*');
 
         // Then I expect an error notification to be displayed that tells me this ROLE length is not allowed
-        AclManagementSteps.getFieldError().contains('Too short');
+        AclManagementSteps.getFieldError().contains('Too short or not "*"');
+    });
+
+    it('should allow creating a new rule if CUSTOM ROLE is a wildcard (*)', () => {
+        // Given I have a defined number of rules at the beginning
+        AclManagementSteps.getAclRules().should('have.length', 5);
+        // When I am on "ACL Management" page and create a new rule with a CUSTOM ROLE of 1 symbol (a wildcard)
+        AclManagementSteps.addRuleInBeginning();
+        AclManagementSteps.selectPolicy(0, 'allow');
+        AclManagementSteps.fillRole(0, '*');
+        AclManagementSteps.selectOperation(0, 'write');
+        AclManagementSteps.fillSubject(0, '<urn:Mary>');
+        AclManagementSteps.fillPredicate(0, '*');
+        AclManagementSteps.fillObject(0, '*');
+        AclManagementSteps.fillContext(0, '*');
+
+        // Then I expect to not see an error
+        AclManagementSteps.getFieldError().should('not.exist');
+        // When I save the rule
+        AclManagementSteps.saveRule(0);
+        // When I save the rules
+        AclManagementSteps.saveAcl();
+        // Then the table should contain the new rule
+        AclManagementSteps.getSavedRoleField(0).should('contain', '*');
+        AclManagementSteps.getAclRules().should('have.length', 6);
     });
 
     it('should show message if role prefix is CUSTOM_', () => {
         // When I am on "ACL Management" page and create a new rule with a CUSTOM_ prefix
         AclManagementSteps.addRuleInBeginning();
         AclManagementSteps.selectPolicy(0, 'allow');
-        AclManagementSteps.fillRole(0, 'CUSTOM_ROLE_FOO');
+        // When I enter the prefix in lowercase, the logic should still detect it
+        AclManagementSteps.fillRole(0, 'custom_ROLE_FOO');
 
         // Then I expect the prefix warning to appear
         AclManagementSteps.getPrefixWarning(0).should('be.visible');

package/integration/setup/plugins.spec.js

@@ -12,6 +12,8 @@ describe('Plugins view', () => {
         cy.presetRepository(repositoryId);
         cy.initializeRepository(repositoryId);
         LicenseStubs.spyGetLicense();
+        PluginsStubs.spyPluginsGet(repositoryId);
+        PluginsSteps.visit();
     });
 
     afterEach(() => {
@@ -19,8 +21,6 @@ describe('Plugins view', () => {
     });
 
     it('Should allow to enable and disable the plugins', () => {
-        PluginsStubs.spyPluginsGet(repositoryId);
-        PluginsSteps.visit();
         cy.wait('@get-license');
         cy.wait('@get-plugins');
         PluginsSteps.waitUntilPluginsPageIsLoaded();
@@ -31,9 +31,10 @@ describe('Plugins view', () => {
         PluginsSteps.getPluginSwitchField('history').should('be.checked');
         PluginsSteps.togglePlugin('history');
 
-        PluginsSteps.visit();
+        cy.reload();
         cy.wait('@get-plugins');
-
+        PluginsSteps.waitUntilPluginsPageIsLoaded();
+        PluginsSteps.getPluginsView().should('be.visible');
         PluginsSteps.getPluginByName('history').should('be.visible');
         PluginsSteps.getPluginSwitchField('history').should('not.be.checked');
     });

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
     "name": "graphdb-workbench-tests",
-    "version": "2.8.1-TR1",
+    "version": "2.8.2-TR1",
     "lockfileVersion": 3,
     "requires": true,
     "packages": {
         "": {
             "name": "graphdb-workbench-tests",
-            "version": "2.8.1-TR1",
+            "version": "2.8.2-TR1",
             "license": "Apache-2.0",
             "devDependencies": {
                 "cypress": "^13.3.1",

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "graphdb-workbench-tests",
-    "version": "2.8.1-TR1",
+    "version": "2.8.2-TR1",
     "description": "Cypress tests for GraphDB workbench",
     "scripts": {
         "prepack": "npm shrinkwrap",

package/steps/repository-steps.js

@@ -85,6 +85,16 @@ export class RepositorySteps {
         RepositorySteps.clickRepositoryIcon(repositoryId, '.repository-actions .restart-repository-btn');
     }
 
+    static getEditViewRestartButton() {
+        return cy.get('#restartRepo');
+    }
+
+    static getRepositoryRestartButton(repositoryId) {
+        return RepositorySteps.getRepositoryFromList(repositoryId)
+            .should('be.visible')
+            .find('.repository-actions .restart-repository-btn');
+    }
+
     static createRepository() {
         RepositorySteps.getCreateRepositoryButton().click();
     }
@@ -265,6 +275,10 @@ export class RepositorySteps {
         RepositorySteps.getEditSparqlInstanceBtn(row).click();
     }
 
+    static getRestartRemoteRepoButton(row) {
+        return this.getRemoteGraphDBTable().eq(row).find('.repository-actions .restart-repository-btn');
+    }
+
     static getRemoteGraphDBTable() {
         return cy.get('#wb-locations-locationInGetRemoteLocations');
     }