Skip to content

Small fixes, fix inheritance - #2269

Merged
DavertMik merged 10 commits into
codeceptjs:codeceptjs-v3.0from
potapovDim:small_fixes
Mar 27, 2020
Merged

Small fixes, fix inheritance#2269
DavertMik merged 10 commits into
codeceptjs:codeceptjs-v3.0from
potapovDim:small_fixes

Conversation

@potapovDim

Copy link
Copy Markdown
Contributor

Motivation/Description of the PR

  • Fix inheritance

Type of change

  • [ x] 💅 Polish code

Checklist:

  • [ x] Lint checking (Run npm run lint)
  • [ x] Local tests are passed (Run npm test)

s-syam and others added 4 commits February 25, 2020 12:08
Earlier it seems like it was clicking the first element. Now it clicks the first visible element,
…on (codeceptjs#2228)

* add missing JSDoc parameter for seeInPopup method, update documentation

* refactor duplicated JSDoc of seeInSource method to mustache file
Edits update to new generator version, minor typos, clarifications.

Otherwise good overview docs!
@potapovDim

Copy link
Copy Markdown
Contributor Author

@DavertMik Hey, should i remove this PR?

@potapovDim
potapovDim force-pushed the small_fixes branch 3 times, most recently from c8b13db to ce0c950 Compare March 17, 2020 18:38
* remove let, add const

* fix import error
Comment thread lib/codecept.js Outdated
Comment thread lib/command/configMigrate.js Outdated
Comment thread lib/command/run-multiple/collection.js Outdated
Comment thread lib/command/workers/runTests.js Outdated
Comment thread lib/command/workers/runTests.js Outdated
Comment thread lib/command/workers/runTests.js Outdated

@potapovDim potapovDim left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@DavertMik

Copy link
Copy Markdown
Contributor

Codacy Here is an overview of what got changed by this pull request:

Clones added
============
- lib/helper/WebDriverIO.js  1
- lib/helper/WebDriver.js  3
         

Clones removed
==============
+ lib/command/run.js  -1
+ test/helper/Playwright_test.js  -1
+ test/runner/dry_run_test.js  -1
+ test/helper/Puppeteer_test.js  -1
         

See the complete overview on Codacy

@potapovDim
potapovDim requested a review from Vorobeyko March 19, 2020 19:31
@DavertMik
DavertMik changed the base branch from master to codeceptjs-v3.0 March 24, 2020 18:45
@DavertMik
DavertMik merged commit dc86bc1 into codeceptjs:codeceptjs-v3.0 Mar 27, 2020
@potapovDim
potapovDim deleted the small_fixes branch April 19, 2020 07:02
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.

8 participants