Skip to content

Conversation

@boysim
Copy link
Owner

@boysim boysim commented Oct 31, 2017

Leave a comment

PieterGit and others added 30 commits September 14, 2017 19:16
…ng to 25 sec.

```
not ok 1 admintools "before all" hook
  Error: timeout of 20000ms exceeded. Ensure the done() callback is being called in this test.
      at tests/admintools.test.js:177:7
      at Object.done (node_modules/benv/index.js:32:21)
      at process.nextTick (node_modules/jsdom/lib/old-api.js:347:18)
      at _combinedTickCallback (internal/process/next_tick.js:131:7)
      at process._tickCallback (internal/process/next_tick.js:180:9)

not ok 203 Profile editor "before each" hook for "should produce some html"
  Error: timeout of 20000ms exceeded. Ensure the done() callback is being called in this test.
      at tests/fixtures/headless.js:160:7
      at Object.module.exports.setup (node_modules/benv/index.js:24:56)
      at Function.init [as setup] (tests/fixtures/headless.js:16:10)
      at Context.<anonymous> (tests/profileeditor.test.js:94:14)
```
Somehow the NONE.svg ended up with an incorrect rectangle shape; fixed
to match the aspect ratio of the other symbols.
Having the regex require integers means that only the numeric keypad
will pop on iOS. This prevents the entry of negative values, which is
pretty critical for temp basals.
Remove pattern regex from basal change field
MilosKozak and others added 29 commits October 16, 2017 22:06
handle AAPS profile switches well
…erage_renderer

Adding some test coverage for /lib/client/renderer.js
do not upload temporary profiles from store
…ote-monitor into travis_improve

Update to nightscout 0.10.1 and upgrade nodejs to 8.7.0 and npm update
See PR: #2967
…o 30 sec., just to be sure. I seem to work once with Node 8.7.0, but the test did a timeout once on Node 8.5.0 on Travis.
```
not ok 188 pluginbase "before each" hook for "does stuff"
 Error: timeout of 30000ms exceeded. Ensure the done() callback is being called in this test.
```
update to nodejs 8.8.0 and npm update
decrease timeouts to detect regressions faster in future
set travis container to dist: trusty and give sudo: false a try
@boysim boysim merged commit 5b0d683 into boysim:dev Oct 31, 2017
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.

7 participants