Closed (fixed)
Project:
Behat UI
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2016 at 13:37 UTC
Updated:
29 Dec 2019 at 00:06 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
rajab natshah+1 I'm with that .. We do need it very much.
Comment #3
rajab natshahComment #4
rajab natshahComment #5
jozzy_a commentedRajab I can help porting this module over to D8. Have you already started the porting?
Joz
Comment #6
rajab natshahYes, Joz
I will commit a new branch for that.
Any help will be rewarded.
It's good to see that we do have an interest in the module,
I only had just granted an access to commit #2823411: Offering to maintain Behat UI
I like to play in a group, Have collaborative work to have the Job done!!!
Thank you :)
Comment #7
jozzy_a commentedHappy to help :)
Have you managed to push up a D8 branch yet?
Joz
Comment #8
rajab natshahActive working on it.
Comment #9
ericmaster commentedIs there any work in progress for this? I'm willing to help in porting this module to D8 as I think it will provide some value added to my client.
Comment #11
rajab natshahHi Eric, Joz,
I have pushed my progress work.
Still many parts to work on.
Comment #12
rajab natshahComment #13
rajab natshahComment #14
it-cru@RajabNatshah : I have tested yesterday the current 8.x-1.x-dev branch and many things throws fatal errors and module does not work for D8. My idea was to build a behat_reports module, but then I found this module which could also work as base for my idea.
Do you plan currently any work on the D8 port? If not I could possible refactor the current D8 code to get it working.
Comment #15
rajab natshahHi Steffen,
Sorry, I did not have time to continue work on the port.
Any help is welcome for sure.
Karim and Caio are very collaborative and welcoming :)
Comment #16
it-cruHello together,
I've started working on a patch. I've currently separated the settings and running tests form. On the create test form I currently get a fatal error which I've doesn't fixed yet.
New settings form is working fine, but some settings like directory settings are missing yet.
Run tests form I will try to fix in the next days.
When running tests and settings form are working fine I attach the patch for review and discussion.
Comment #17
it-cruFirst throw of my patch to fix broken D8 port. Running tests with behat pretty formatter is working fine. For HTML formatter I have used "elkan/behatformatter": "^1.0" because behat 3 does not provide a HTML formatter any more. HTML formatter is disabled by default on running tests form.
Not working yet:
The patch is very big. Perhaps it makes more sense to join as co-maintainer for D8 branch of the behat_ui module?
Comment #18
it-cruComment #20
it-cruFix missing or wrong stuff of #17 patch.
Comment #22
rajab natshahThank you Steffen,
Committed ... :)
We will take any work from your point of work
I will be testing this module in
Cucumber Management System
If you like Steffen we could collaborate work in the Cucumber Drupal profile project too
:)
Comment #23
rajab natshahOh Seems that we do have a new patch
Comment #25
rajab natshahCommitted last patch #20
We will move foreword with more testing from last commit.
Before we go for Behat UI 8.x-1.0-alpha1
Comment #26
rajab natshahAfter testing:
Not sure yet, we could have a behat.yml auto locater and Behat auto dedicated process.
An alert message could do the work. if we do not want to run not selected gherkin scripts.
Comment #27
it-cru@RajabNatshah: I think a general behat.yml auto locator is not so good, because the behat config file could be located in very different places in projects. For example I have a behat.yml in every sites/example.com directory in my multisite setup for local testing of the specific site.
Do you think we should open separate issues for D8 branch for fixing run tests, new tests, settings form and other stuff?
I currently will try to figure out how to run only a special feature / suite during run tests.
Comment #28
rajab natshahYes Steffen with you on that.
Just to let the user know that behat.yml is not set yet and a link in the message to set it.
For sure we could create more issues. for all new coming ideas of fixes.
I committed your work, so that we could continue work from your point of work progress.
Thank you :)
Comment #29
rajab natshahComment #30
rajab natshah