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

Incorrect callsite+callstack for failed assertion in a method of some class #1267

Closed
AlexanderMoskovkin opened this issue Feb 23, 2017 · 4 comments
Labels
AREA: server STATE: Auto-locked An issue has been automatically locked by the Lock bot. SYSTEM: reporting TYPE: bug The described behavior is considered as wrong (bug).
Milestone

Comments

@AlexanderMoskovkin
Copy link
Contributor

Are you requesting a feature or reporting a bug?

bug

What is the current behavior?

      AssertionError: expected false to be truthy

      Browser: Chrome 56.0.2924 / Windows 10 0.0.0

          6 |    }
          7 |}
          8 |
          9 |async function fn (t) {
         10 |    // any code
       > 11 |}
         12 |
         13 |test('test', async t => {
         14 |    const page = new Page();
         15 |
         16 |    await page.expect(t);

         at <anonymous> (c:\temp\example\test.js:11:276)
         at Page.expect (c:\temp\example\test.js:11:283)
         at <anonymous> (c:\temp\example\test.js:16:20)
         at test (c:\temp\example\test.js:17:44)

What is the expected behavior?

It should be an error on the line 5.

Provide the test code and the tested page URL (if applicable)

Here is a simple test to reproduce the problem:

fixture `f`;

class Page {
    async expect (t) {
        await t.expect(false).ok();
    }
}

async function fn (t) {
    // any code
}

test('test', async t => {
    const page = new Page();

    await page.expect(t);
});

Specify your

  • operating system: Win 10
  • testcafe version: 0.14.0-alpha1
  • node.js version: v6.9.1
@AlexanderMoskovkin AlexanderMoskovkin added AREA: server SYSTEM: reporting TYPE: bug The described behavior is considered as wrong (bug). labels Feb 23, 2017
@AlexanderMoskovkin AlexanderMoskovkin added this to the Sprint #5 milestone Feb 23, 2017
@inikulin
Copy link
Contributor

@AlexanderMoskovkin Is it regression after #1226?

@AlexanderMoskovkin
Copy link
Contributor Author

No, it's reproduced with the 0.13.0 version too. (#1226 fix added in 0.14.0-alpha1)

@georgiy-abbasov
Copy link
Contributor

@georgiy-abbasov georgiy-abbasov modified the milestones: Sprint #6, Sprint #5 Mar 15, 2017
georgiy-abbasov pushed a commit to georgiy-abbasov/testcafe-phoenix that referenced this issue Mar 29, 2017
georgiy-abbasov pushed a commit to georgiy-abbasov/testcafe-phoenix that referenced this issue Mar 29, 2017
georgiy-abbasov pushed a commit to georgiy-abbasov/testcafe-phoenix that referenced this issue Mar 30, 2017
@lock
Copy link

lock bot commented Mar 29, 2019

This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests. We recommend you ask TestCafe API, usage and configuration inquiries on StackOverflow.

@lock lock bot added the STATE: Auto-locked An issue has been automatically locked by the Lock bot. label Mar 29, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Mar 29, 2019
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this issue Dec 18, 2019
…vExpress#1369)

* Get rid of "retainLines" compiler option (closes DevExpress#1267)

* Update custom "callsite-record" module

* update callsite-record to 4.0.0

* comment added
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AREA: server STATE: Auto-locked An issue has been automatically locked by the Lock bot. SYSTEM: reporting TYPE: bug The described behavior is considered as wrong (bug).
Projects
None yet
Development

No branches or pull requests

3 participants