Skip to content

resolution for Not able to override step.provider.pkg property for data driven scenario#155

Open
infoneershalin wants to merge 5 commits intoqaf-tm:masterfrom
infoneershalin:master
Open

resolution for Not able to override step.provider.pkg property for data driven scenario#155
infoneershalin wants to merge 5 commits intoqaf-tm:masterfrom
infoneershalin:master

Conversation

@infoneershalin
Copy link
Copy Markdown

@infoneershalin infoneershalin commented Nov 28, 2017

resolution for #154

we were using initStep only when step is not yet initiated.
we need to re init the step in case fileName is different as step.provider.pkg has changed

@infoneershalin
Copy link
Copy Markdown
Author

Will add Unit Testcases soon.

Copy link
Copy Markdown
Author

@infoneershalin infoneershalin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cjayswal please review this Pull request

@infoneershalin infoneershalin changed the title resolution for #154 resolution for Not able to override step.provider.pkg property for data driven scenario Nov 28, 2017
@cjayswal
Copy link
Copy Markdown
Contributor

Did you understand the issue? Please share the test case. I am afraid that solution will have impact on performance.

@infoneershalin
Copy link
Copy Markdown
Author

infoneershalin commented Nov 28, 2017

Yes, DataDrivenScenario.java creates proxy steps and pass it to execute method of Scenario.java
As proxy steps are already initiated before any step executed, the change in step.provider.pkg property by subsequent step does not get reflected.
Where as for non DataDrivenScenario steps get initiated one by one and hence change in step.provider.pkg get reflected properly
attaching TestProject_ISSUE_154.zip here

@cjayswal
Copy link
Copy Markdown
Contributor

You can check by commenting step initialization to have lazy initialization in DD scenario

@infoneershalin
Copy link
Copy Markdown
Author

infoneershalin commented Nov 28, 2017 via email

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants