mitchell852 commented on a change in pull request #3949: Fixes broken regions
and phys location UI tests
URL: https://github.com/apache/trafficcontrol/pull/3949#discussion_r330258479
##########
File path: traffic_portal/test/end_to_end/divisions/divisions-spec.js
##########
@@ -67,11 +67,4 @@ describe('Traffic Portal Divisions Test Suite', function() {
expect(pageData.name.getText() === myNewDiv.name + ' updated');
});
- it('should delete the new division', function() {
- console.log('Deleting the new division: ' + myNewDiv.name + '
updated');
- pageData.deleteButton.click();
- pageData.confirmWithNameInput.sendKeys(myNewDiv.name + '
updated');
Review comment:
no, the delete works fine. the problem is that for these tests to work, you
can't delete certain things.
i.e. a division is needed to create a region which is needed to create a
phys location. therefore, divisions-spec and regions-spec can't have a 'delete'
test.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
