search

TestArmada / magellan-saucelabs-executor

magellan test executor for saucelabs
MIT License
2 stars 2 forks source link

Remove locks #27

Closed archlichking closed 6 years ago

archlichking commented 6 years ago

This is to support magellan@11 release. Now magellan talks to locks via a global resource strategy so that such resource can be applied on all executors.

codecov-io commented 6 years ago

Codecov Report

Merging #27 into master will increase coverage by 10.36%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #27       +/-   ##
===========================================
+ Coverage   85.31%   95.68%   +10.36%     
===========================================
  Files          10        9        -1     
  Lines         613      371      -242     
  Branches      116        0      -116     
===========================================
- Hits          523      355      -168     
+ Misses         90       16       -74
Impacted Files Coverage Δ
src/configuration.js 100% <100%> (ø)
src/profile.js 94.28% <100%> (ø)
src/settings.js 100% <100%> (ø)
src/logger.js 100% <100%> (ø)
src/util/guid.js 100% <100%> (ø)
src/global_analytics.js 100% <100%> (ø)
src/executor.js 90.9% <100%> (ø)
src/tunnel.js 96.55% <100%> (ø)
lib/configuration.js
lib/locks_socket_api.js
... and 17 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2d748e5...5889aae. Read the comment docs.