Merge pull request #4465 from BryceLTaylor/Move-project-page-tests

Move project page tests

.travis.yml

@@ -48,6 +48,11 @@ env:
   - PROJECT_HOST_VAR=PROJECT_HOST_$TRAVIS_BRANCH
   - PROJECT_HOST=${!PROJECT_HOST_VAR}
   - PROJECT_HOST=${PROJECT_HOST:-$PROJECT_HOST_STAGING}
+  - TEST_PROJECT_ID_master=414835599
+  - TEST_PROJECT_ID_STAGING=1300006196
+  - TEST_PROJECT_ID_VAR=TEST_PROJECT_ID_$TRAVIS_BRANCH
+  - TEST_PROJECT_ID=${!TEST_PROJECT_ID_VAR}
+  - TEST_PROJECT_ID=${TEST_PROJECT_ID:-$TEST_PROJECT_ID_STAGING}
   - STATIC_HOST_master=https://cdn2.scratch.mit.edu
   - STATIC_HOST_STAGING=https://cdn.scratch.ly
   - STATIC_HOST_VAR=STATIC_HOST_$TRAVIS_BRANCH

README.md

@@ -215,11 +215,13 @@ so for the time being our tests run using both.
 To run all integration tests from the command-line:
 
 ```bash
-SMOKE_USERNAME=username SMOKE_PASSWORD=password ROOT_URL=https://scratch.mit.edu npm run test:integration
+SMOKE_USERNAME=username SMOKE_PASSWORD=password ROOT_URL=https://scratch.mit.edu TEST_PROJECT_ID=1 npm run test:integration
 ```
 
 Both the TAP and Jest tests use the same username and password.  The Jest tests will also use the the username you give with a 1 (soon to be higher numbers as well) appended to the end of it.  So if you use the username "test" it will also use the username "test1."  Make sure you have created accounts with this pattern and use the same password for all accounts involved.
 
+The project page tests require a project id included as an environment variable to pass.  The project must be shared and must have at least one remix.  At this time, the project does not need to be owned by one of the test users, but that is likely to change.
+
 To run a single file from the command-line using Jest:
 
 ```bash

test/integration-legacy/smoke-testing/test_project_page.js

@@ -1,61 +0,0 @@
-const SeleniumHelper = require('../selenium-helpers.js');
-const helper = new SeleniumHelper();
-
-var tap = require('tap');
-const test = tap.test;
-
-const driver = helper.buildDriver('www-smoke test_sign_in_out_homepage');
-
-const {
-    clickText,
-    clickXpath,
-    dragFromXpathToXpath,
-    findByXpath,
-    waitUntilGone
-} = helper;
-
-const rootUrl = process.env.ROOT_URL || 'https://scratch.ly';
-const projectId = 1;
-const projectUrl = `${rootUrl}/projects/${projectId}`;
-
-tap.plan(3);
-
-tap.tearDown(function () {
-    driver.quit();
-});
-
-tap.beforeEach(function () {
-    return driver.get(projectUrl);
-});
-
-test('Find fullscreen button', {skip: true}, t => {
-    findByXpath('//div[starts-with(@class, "loader_background")]')
-        .then(el => waitUntilGone(el))
-        .then(() => clickXpath('//div[starts-with(@class, "stage_green-flag-overlay")]'))
-        .then(() => clickXpath('//img[contains(@alt, "Enter full screen mode")]'))
-        .then(() => t.end());
-});
-
-test('Open and close Copy Link modal', {skip: true}, t => {
-    findByXpath('//div[starts-with(@class, "loader_background")]')
-        .then(el => waitUntilGone(el))
-        .then(() => clickText('Copy Link'))
-        .then(() => clickXpath('//div[contains(@class, "social-label-title")]'))
-        .then(() => clickXpath('//img[contains(@alt, "close-icon")]'))
-        .then(() => clickXpath('//img[contains(@alt, "Enter full screen mode")]'))
-        .then(() => t.end());
-});
-
-test('Dragging out of modal should not close modal', {skip: true}, t => {
-    findByXpath('//div[starts-with(@class, "loader_background")]')
-        .then(el => waitUntilGone(el))
-        .then(() => clickXpath('//div[starts-with(@class, "stage_green-flag-overlay")]'))
-        .then(() => clickText('Copy Link'))
-        .then(() => clickXpath('//div[contains(@class, "social-label-title")]'))
-        .then(() => dragFromXpathToXpath(
-            '//div[contains(@class, "social-label-title")]',
-            '//li[contains(@class, "logo")]'
-        ))
-        .then(() => clickXpath('//div[contains(@class, "social-label-title")]'))
-        .then(() => t.end());
-});

test/integration/project-page.test.js

@@ -0,0 +1,74 @@
+const SeleniumHelper = require('./selenium-helpers.js');
+
+const {
+    findByXpath,
+    clickXpath,
+    buildDriver
+} = new SeleniumHelper();
+
+let remote = process.env.SMOKE_REMOTE || false;
+let rootUrl = process.env.ROOT_URL || 'https://scratch.ly';
+let projectId = process.env.TEST_PROJECT_ID || 1300006196;
+let projectUrl = rootUrl + '/projects/' + projectId;
+
+if (remote){
+    jest.setTimeout(60000);
+} else {
+    jest.setTimeout(10000);
+}
+
+let driver;
+
+describe('www-integration project-page signed out', () => {
+    beforeAll(async () => {
+        // expect(projectUrl).toBe(defined);
+        driver = await buildDriver('www-integration project-page signed out');
+        await driver.get(rootUrl);
+    });
+
+    beforeEach(async () => {
+        await driver.get(projectUrl);
+        let gfOverlay = await findByXpath('//div[@class="stage-wrapper_stage-wrapper_2bejr box_box_2jjDp"]');
+        await gfOverlay.isDisplayed();
+    });
+
+    afterAll(async () => await driver.quit());
+
+    // LOGGED OUT TESTS
+    
+    test('Find fullscreen button', async () => {
+        await clickXpath('//div[starts-with(@class, "stage_green-flag-overlay")]');
+        await clickXpath('//img[contains(@alt, "Enter full screen mode")]');
+        let fullscreenGui = await findByXpath('//div[@class="guiPlayer fullscreen"]');
+        let guiVisible = await fullscreenGui.isDisplayed();
+        await expect(guiVisible).toBe(true);
+    });
+
+    test('Open Copy Link modal', async () => {
+        await clickXpath('//button[@class="button action-button copy-link-button"]');
+        let projectLink = await findByXpath('//input[@name="link"]');
+        let linkValue = await projectLink.getAttribute('value');
+        await expect(linkValue).toEqual(projectUrl);
+    });
+
+    test('Click Username to go to profile page', async ()=> {
+        await clickXpath('//div[@class="title"]/a');
+        let userContent = await findByXpath('//div[@class="user-content"]');
+        let contentVisible = await userContent.isDisplayed();
+        await expect(contentVisible).toBe(true);
+    });
+
+    test('click See Inside to go to the editor', async ()=> {
+        await clickXpath('//button[@class="button button see-inside-button"]');
+        let infoArea = await findByXpath('//div[@class="sprite-info_sprite-info_3EyZh box_box_2jjDp"]');
+        let areaVisible = await infoArea.isDisplayed();
+        await expect(areaVisible).toBe(true);
+    });
+
+    test('click View All remixes takes you to remix page', async ()=> {
+        await clickXpath('//div[@class="list-header-link"]');
+        let originalLink = await findByXpath('//h2/a');
+        let link = await originalLink.getAttribute('href');
+        await expect(link).toEqual(rootUrl + '/projects/' + projectId + '/');
+    });
+});