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

Error when fetch used with data-url #2865

Closed
mixtur opened this issue Sep 14, 2018 · 4 comments
Closed

Error when fetch used with data-url #2865

mixtur opened this issue Sep 14, 2018 · 4 comments
Assignees
Labels
AREA: client COMPLEXITY: easy STATE: Auto-locked An issue has been automatically locked by the Lock bot. SYSTEM: hammerhead TYPE: bug The described behavior is considered as wrong (bug).

Comments

@mixtur
Copy link

mixtur commented Sep 14, 2018

Are you requesting a feature or reporting a bug?

bug

What is the current behavior?

testcafe throws an error
Uncaught TypeError: Cannot read property 'destUrl' of null

What is the expected behavior?

no error

How would you reproduce the current behavior (if this is a bug)?

use fetch with data-url in tested code.
For example:

<script>
    fetch('data:text/plain,foo')
        .then(x => x.text())
        .then(console.log);
</script>

Any test will fail including empty
For example:

fixture`bug`
    .page`./index.html`;

test('empty test', async () => {});

Specify your

  • operating system: Ubuntu 16.04
  • testcafe version: 0.22.0
  • node.js version: 10.9.0
@miherlosev miherlosev self-assigned this Sep 14, 2018
@miherlosev miherlosev added TYPE: enhancement The accepted proposal for future implementation. AREA: client SYSTEM: hammerhead labels Sep 14, 2018
@miherlosev miherlosev added this to the Planned milestone Sep 14, 2018
@miherlosev
Copy link
Collaborator

Hi @mixtur

At present, TestCafe does not support calling fetch with data urls. I've marked this issue as enhancement.

@LavrovArtem LavrovArtem added TYPE: bug The described behavior is considered as wrong (bug). COMPLEXITY: easy and removed TYPE: enhancement The accepted proposal for future implementation. labels Sep 17, 2018
@LavrovArtem LavrovArtem modified the milestones: Planned, Sprint #17 Sep 17, 2018
@AndreyBelym AndreyBelym modified the milestones: Sprint #17, Sprint #18 Sep 17, 2018
LavrovArtem added a commit to LavrovArtem/testcafe-hammerhead that referenced this issue Sep 24, 2018
LavrovArtem added a commit to DevExpress/testcafe-hammerhead that referenced this issue Sep 24, 2018
@miherlosev miherlosev modified the milestones: Sprint #18, Sprint #19 Oct 1, 2018
@miherlosev
Copy link
Collaborator

The fix was published in [email protected]

@mixtur
Copy link
Author

mixtur commented Oct 3, 2018

thx

@lock
Copy link

lock bot commented Mar 28, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs or feature requests. For TestCafe API, usage and configuration inquiries, we recommend asking them on StackOverflow.

@lock lock bot added the STATE: Auto-locked An issue has been automatically locked by the Lock bot. label Mar 28, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Mar 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AREA: client COMPLEXITY: easy STATE: Auto-locked An issue has been automatically locked by the Lock bot. SYSTEM: hammerhead TYPE: bug The described behavior is considered as wrong (bug).
Projects
None yet
Development

No branches or pull requests

4 participants