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

Dalek action (e.g. type) does not cancel if element is not found #86

Open
izocan opened this issue Apr 17, 2014 · 8 comments
Open

Dalek action (e.g. type) does not cancel if element is not found #86

izocan opened this issue Apr 17, 2014 · 8 comments
Assignees

Comments

@izocan
Copy link

izocan commented Apr 17, 2014

Hey I have a textfield like this:

<input id="name" class="ember-view ember-text-field form-control" placeholder="Required" maxlength="20" type="text" data-testid="textfieldTerminalId">

If I use the dalek action type:

.type('[data-testid="txtfieldTerminalId"]', '12345678')

So when misspelling the attribute value, dalekjs seems to be looking for this element forever and does not cancel the current action. (i assume the action commands have no timeout).

I need the actions which cannot be executed cancelled, due to the fact that I want to automate my tests and dont want my whole test set depend on every type action I use in my test code. Is there any workaround or release which considers this issue?

Versions:
Win7
dalek-cli 0.0.4
dalekjs local install: 0.0.8

cheers izocan

@asciidisco
Copy link
Member

I do not have a workaround for you, sorry. This is really a nasty bug, related to #2

@izocan
Copy link
Author

izocan commented Apr 17, 2014

-.- will you treat this issue in the next release? i can work with this error now, but in future the whole team will write tests and the whole stuff will move towards automation. (so it does now but we dont have much tests and things are easier to fix).
Cheers

@asciidisco
Copy link
Member

I will try, but I wouldn't rely on it. If you wan't I could assist you when trying to get rid of this one. You know, Open Source and stuff ;)

@izocan
Copy link
Author

izocan commented Apr 28, 2014

hehe :) by the way you wanted to send me some low-level bugs to fix. if this is a bug i can handle why not ;). This would be the first time contributing to OpenSource. I can check it after i have writteen my bachelors thesis.
AND check your public email please if you can :) cheers

@asciidisco
Copy link
Member

Hey,

please ping me, when you are done with your thesis, we can then check together which bugs are low hanging fruits for you to tackle.

@izocan
Copy link
Author

izocan commented Jun 5, 2014

hey dude. im finished with the thesis. I need an urgent fix for this issue. I need at least something like a timeout. So the test should progress. I already tried so many workarounds. They worked locally during setting up the testsuites. But now we use the tests productive for our nightly builds. And the workarounds seem not to work right and the tests never progress further.

Could you assist me while fixing this bug??

cheers izocan.

@Tuurlijk
Copy link

Tuurlijk commented Jun 6, 2014

I would love to see this solved as well. It's a real dealbreaker for me. Since my test now 'hangs' when the page does not contain the expected elements (v 0.0.8).

@asciidisco asciidisco added the bug label Aug 13, 2014
@asciidisco asciidisco self-assigned this Aug 13, 2014
@asciidisco
Copy link
Member

We will tackle this in the "super shiny new major rewrite" version, which does not work at the moment but estimations are that we will release an early version in October.

Follow this repo to see the current status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants