Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

helper function and integration test #1540

Merged

Conversation

gluneau
Copy link
Contributor

@gluneau gluneau commented Feb 10, 2016

No description provided.

@gluneau gluneau added the Testing PR contains test plan or automated test code (or config files for Travis) label Feb 10, 2016
@gluneau gluneau added this to the 16.04 milestone Feb 10, 2016
@ted-strauss-K1
Copy link
Contributor


                                    _.--"""--,
                                    .'          `\
  .-""""""-.                      .'              |
 /          '.                   /            .-._/
|             `.                |             |
 \              \          .-._ |          _   \
  `""'-.         \_.-.     \   `          ( \__/
        |             )     '=.       .,   \  
       /             (         \     /  \  /
     /`               `\        |   /    `'
     '..-`\        _.-. `\ _.__/   .=.
          |  _    / \  '.-`    `-.'  /
          \_/ |  |   './ _     _  \.'
               '-'    | /       \ |  
                      |  .-. .-.  |
                      \ / o| |o \ /
                       |   / \   |
                      / `"`   `"` \
                     /             \
                    | '._.'         \
                    |  /             |
                     \ |             |
                      ||    _    _   /
                      /|\  (_\  /_) /
                       \ \'._  ` '_.'
                       `""` `"""` 

@xlecours xlecours self-assigned this Feb 19, 2016
@xlecours
Copy link
Contributor

Travis show this error :1) CreateTimepointTestIntegrationTest::testCreateTimepointDoespageLoad
Failed asserting that 'Console Output' contains "Create Time Point".

You have an error in your page. I suggestion you look at the full error with something like :
var_dump( $this->webDriver->findElement(WebDriverBy::xPath('//div[@role="alert"]'))->getText() );

@gluneau gluneau force-pushed the tests-cases-for-create_timepoint-module branch from ded264a to 1d41250 Compare February 25, 2016 20:15
@codecov-io
Copy link

Current coverage is 15.03%

Merging #1540 into 16.04-dev will not affect coverage as of 595747d

@@            16.04-dev   #1540   diff @@
=========================================
  Files             108     108       
  Stmts           18273   18270     -3
  Branches            0       0       
  Methods          1025    1025       
=========================================
- Hit              2748    2746     -2
  Partial             0       0       
+ Missed          15525   15524     -1

Review entire Coverage Diff as of 595747d

Powered by Codecov. Updated on successful CI builds.

* text appears in the body.
*
* @return void

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be better to uncomment this test and add a markTestIncomplete() or markTestSkipped()

@gluneau gluneau added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Feb 25, 2016
samirdas added a commit that referenced this pull request Feb 26, 2016
…t-module

helper function and integration test
@samirdas samirdas merged commit 36b68fd into aces:16.04-dev Feb 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Testing PR contains test plan or automated test code (or config files for Travis)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants