-
Notifications
You must be signed in to change notification settings - Fork 17
Port the paper-element tests! #6
Comments
Guys, does anyone started working on this one? |
I'd like to help, but I might not be able to get to it until the weekend. There's 30 so of them, so maybe just pick 5-10 at a time and post them here so we know who's working on what? |
sounds good! |
If its easier I am also more than happy to split these up into a few different tasks |
Ok, I'll take first 6 in the /src/paper-*
These elements are skipped:
|
@jakemac53 Thanks, but I think this will work fine. @vsheyanov Ok. I'll take these:
|
next batch
|
@jakemac53 should we create a pull request when a batch of test is ready or better to create one in the end? |
@vsheyanov however you would prefer |
@jakemac53 looks like paper-submenu is missing in polymer-dart, but exists in original repo. What should be done in this case? |
@jakemac53 also, could you clarify, is it normally, that constants from JS elements don't exist in generated dart sources? For instance |
@jakemac53 I'm looking at the one of the tests for paper-icon-button that's failing. the dom
this test is failing
but I can find iron-meta
Bug in polymer? |
taking the additional tests in the paper-input repo (Added to my original comment) |
@vsheyanov I am working on adding a few other missing elements, and I will make sure paper-submenu is a part of that. In terms of js constants we don't have support for accessing those from dart wrappers today. Usually they are just things used internally, but you could feel free to add an issue if there is a constant which we should actually be exposing. |
@ErikGrimes that does seem like a bug, what about if you just do: expect(b1.$['icon'].querySelector('svg'), isNotNull); |
@vsheyanov paper-submenu has now been added in master |
@jakemac53 I get the same result. The svg element isn't present in the children property either. The DOM above is from using paper-icon-button in a demo application and inspecting via Dartium. The element is setup properly and the queries work in the demo app. I was hoping to reproduce the issue in a gist but it's proving to be a bit of pain. I'm not sure if it's another bug or simply my lack of understanding.
In the above the template is in the children, but can't be found by querySelector. I grabbed it by id from $, but the it turns out the content is null. Any advice on getting the repro to work? Should I merge with the failing test and file a bug pointing to the test? |
@ErikGrimes I would just mark it as skipped and link an issue to investigate, I can take a look at it later. Thanks! |
Picking up:
|
Next batch:
|
|
|
Just a heads up, I'm not sure when I'll get back to this. I upgraded my Mac and now content-shell and dartium don't work with test. Test just times out trying to connect. I filed an issue dart-lang/test#324. So, feel free to pick up the last three from my batch. I won't be doing anything until I get my dev environment sorted out. |
@ErikGrimes no problem thanks for the help and the heads up! |
@jakemac53 You're welcome! Don't merge my last pull request. I think it might have something to do with my problems. I made a fresh clone of dart-lang/polymer_elements and the tests work fine. Tests from other projects work as well. My last change was to common which is included in every test. The odd thing is I swear all the tests before I pushed and they passed. |
@jakemac53 I confirmed it's related to my fork. I cloned my fork of polymer_elements on my Mac at work and it crashed content-shell. |
@ErikGrimes I edited your comment above to remove the remaining items, and I will start working on them:
|
ok, all done it looks like! |
Part of #1. This may need to be split up into multiple sub-tasks.
The text was updated successfully, but these errors were encountered: