Skip to content

Commit

Permalink
OPENEUROPA-1500: Simplify test.
Browse files Browse the repository at this point in the history
  • Loading branch information
@nagyad
nagyad committed Jan 10, 2019
1 parent e615e06 commit 114f7d9
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions tests/features/analytics.feature
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,10 @@ Feature: Webtools Analytics
As the site manager
I need to be able to configure the settings

Background:
Given I am logged in as a user with the "administer webtools analytics" permission

@BackupAnalyticsConfigs
Scenario: Create Webtools Analytics settings
Given I am on "admin/config/system/oe_webtools_analytics"
Given I am logged in as a user with the "administer webtools analytics" permission
And I am on "admin/config/system/oe_webtools_analytics"

This comment has been minimized.

Sign in to view

Copy link
@drupol

drupol Jan 10, 2019

Contributor

Why we do not create such:

And I want to configure the module oe_webtools_analytics

It would makes much more sense business wise, no ?

This comment has been minimized.

Sign in to view

Copy link
@nagyad

nagyad Jan 10, 2019

Author Member

I think the alias is quite alright from business point of view as well. On the other hand I'm following this way to remain consistent with the rest of the components.

This comment has been minimized.

Sign in to view

Copy link
@drupol

drupol Jan 10, 2019

Contributor

Consistency is key, true.
But... if for every ticket, we create 3 sub-tickets, we have an issue.
I propose to tackle those issue "on-the-go" and slowly refine everywhere properly.
Then I should see "Webtools Analytics settings"
When I fill in "Site ID" with "123456"
And I fill in "Site path" with "ec.europa.eu/info"
Expand Down

0 comments on commit 114f7d9

Please sign in to comment.