Merge pull request #426 from ccheraa/tests-rewrite

basic api tests

Author: duncdrum

cypress/fixtures/bad_api.json

@@ -1,11 +1,12 @@
 {
-    "server": {
-        "product-name": "eXist",
-        "build": "20200123123609",
-        "version": "5.2.0",
-        "revision": "c58d04ec45de50e7738489dee072fcc863dc8b1b"
-    },
-    "exist-db": {
-        "compatible-version": "0.0.1"
-    }
+  "version": "0.0.1",
+  "server": {
+    "product-name": "eXist",
+    "build": "20200123123609",
+    "version": "5.2.0",
+    "revision": "c58d04ec45de50e7738489dee072fcc863dc8b1b"
+  },
+  "exist-db": {
+    "compatible-version": null
+  }
 }

cypress/integration/02_api-spec.js

@@ -1,13 +1,42 @@
 /// <reference types="Cypress" />
 
 context('Talking to the api directly', () => {
-    describe('API version', () => {
-        it.skip('should connect with newer api', () => {
-            cy.visit('/')
-            // TODO: Why does this have  UUID is the ID stable, this needs a meaningful selector
-              .get('#d184cace-9938-4ad5-b8df-925a91942661')
-              .should('be.visible')
-        })
-
-    })
+  describe('API version', () => {
+    before(function() {
+      cy.connect();
+      cy.visit('/');
+    });
+    it('should fail to connect with older api', () => {
+      cy.intercept('GET', Cypress.env('API_HOST') + '/exist/restxq/fusiondb/version', { fixture: 'bad_api' });
+      cy.get('.fusion-item').click().then(() => {
+        cy.get('.dialogTitle').should('contain.text', 'New Connection');
+        cy.get('.dialogContent').should('be.visible')
+          .should('contain.text', 'Outdated API "0.0.1"')
+          .should('contain.text', 'You need to update your API to version "0.2.0" or higher');
+        cy.get('.theia-button.main').should('be.visible').click();
+        cy.get('.dialogBlock').should('not.exist');
+      });
+    });
+    it('should connect with newer api', () => {
+      cy.window().then(function(win) {
+        const fetchSpy = cy.spy(win, 'fetch');
+        fetchSpy.withArgs(Cypress.env('API_HOST') + '/exist/restxq/fusiondb/version').as('/version');
+        fetchSpy.withArgs(Cypress.env('API_HOST') + '/exist/restxq/fusiondb/explorer?uri=/').as('/explorer');
+        fetchSpy.withArgs(Cypress.env('API_HOST') + '/exist/restxq/fusiondb/user').as('/user');
+        fetchSpy.withArgs(Cypress.env('API_HOST') + '/exist/restxq/fusiondb/group').as('/group');
+        fetchSpy.withArgs(Cypress.env('API_HOST') + '/exist/restxq/fusiondb/index').as('/index');
+        fetchSpy.withArgs(Cypress.env('API_HOST') + '/exist/restxq/fusiondb/restxq').as('/restxq');
+        cy.get('.fusion-item').click();
+        cy.get('.fusion-view')
+          .should('contain', 'db')
+          .should('contain', 'RestXQ');
+        cy.get('@/version').should('be.called');
+        cy.get('@/explorer').should('be.called');
+        cy.get('@/user').should('be.called');
+        cy.get('@/group').should('be.called');
+        cy.get('@/index').should('be.called');
+        cy.get('@/restxq').should('be.called');
+      });
+    });
+  });
 })

cypress/integration/04_document_spec.js

@@ -4,8 +4,8 @@ context('Document Operations', () => {
   describe('working with tree view', () => {
     before(() => {
       cy.connect()
-      // TODO(DP): might have to improve by adding more before / after hooks to prevent dangling documents
-      // see #400
+        // TODO(DP): might have to improve by adding more before / after hooks to prevent dangling documents
+        // see #400
     })
 
     describe('db context menu', () => {
@@ -15,7 +15,7 @@ context('Document Operations', () => {
           .should('be.visible')
         cy.get('.fusion-item')
           .click()
-        //  all we need is the final part of the node-id attribute
+          //  all we need is the final part of the node-id attribute
         cy.get('[node-id$=db]')
           .rightclick()
           .then(() => {
@@ -27,11 +27,11 @@ context('Document Operations', () => {
               .should('be.visible')
               .click()
           })
-        // (DP): start workaround for #413 
+          // (DP): start workaround for #413 
         cy.get('[node-id$=db]')
           .trigger('mousemove')
           .type('{enter}')
-        // end workaround for #413
+          // end workaround for #413
         cy.get('.ReactVirtualized__Grid')
           .contains('untitled-1')
 
@@ -47,13 +47,13 @@ context('Document Operations', () => {
       // or by using the file menu UI instead
       // see #414
       it('should let users edit new document', () => {
-        cy.get('[node-id$=untitled-1]')
-          .dblclick()
-        cy.get('.view-line')
-          .type('asdf{meta+s}')
-      })
-      // see #414 workaround is to run this after editing and saving the document, 
-      // we should be able to rename before entering content
+          cy.get('[node-id$=untitled-1]')
+            .dblclick()
+          cy.get('.view-line')
+            .type('asdf{meta+s}')
+        })
+        // see #414 workaround is to run this after editing and saving the document, 
+        // we should be able to rename before entering content
       it('should let users rename documents', () => {
         cy.get('[node-id$=untitled-1]')
           .rightclick()
@@ -70,19 +70,19 @@ context('Document Operations', () => {
           .type('{alt+enter}', { force: true })
         cy.get('.dialogTitle')
           .should('contain.text', 'Properties')
-        // rename file -> text.xml
+          // rename file -> text.xml
         cy.get('.value > .theia-input')
           .clear()
           .type('test.xml')
-        // check properties table 
+          // check properties table 
         cy.get('.dialogContent')
           .find('.keys > tr')
           .should('have.length', 11)
           .contains('Media Type')
         cy.get('.dialogContent')
           .find('.keys > tr')
           .contains('Owner')
-        // check permissions table  
+          // check permissions table  
         cy.get('.dialogContent')
           .find('.permissions-editor > tr')
           .should('have.length', 3)
@@ -120,7 +120,7 @@ context('Document Operations', () => {
           .click()
         cy.get('.main')
           .click()
-        // make sure all test files are gone see #400
+          // make sure all test files are gone see #400
         cy.get('[node-id$=untitled-1]')
           .should('not.exist')
         cy.get('[node-id$=test\\.txt]')