• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Galvant / InstrumentKit / 1318 / 1
99%
main: 99%

Build:
Build:
LAST BUILD BRANCH: bf_thorlabs_apt
DEFAULT BRANCH: main
Ran 23 Sep 2020 03:40AM UTC
Files 154
Run time 12s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

23 Sep 2020 03:37AM UTC coverage: 95.009% (+0.8%) from 94.254%
1318.1

push

travis-ci

web-flow
Test suite and BFs for Keithley580 (#263)

* Test suite and BFs for Keithley580

Test suite with full coverage added

**BFs**:
- Removed inaccessible code block
- Replaced a not in `enum` statement with a check of `isinstance`, now
  the test is compatible to the error description that is raised.
- Getting the 'auto' input was previously not possible, fixed the
  respective property getter.
- Fixed typos
- Fixed status word getting such that an IOError can actually be
  raised now if failing to receive a status word after given number of
  tries
- Fixed reading of bytes: same issue as in Keithley195, trying to
  unpack a string with `struct.unpack`. Now consistent and functional.

* Read measurement and status word using `read_raw`

Preferred method over using `query`, since `query` decodes the return,
which then needs to be encoded again prior to unpacking with `struct`.
Now, the return is read using `read_raw`, which can directly be
unpacked.
Note: `query('')` still sends a `:`, which is now sent using `sendcmd`
prior to running `read_raw`. It also strips the last character from
the return, which done here as well.
Updated tests.

* Status word now created according to manual, read in w/o last byte

The status word, according to the manual, contains the termination
character. It is unclear if this is the same as the end line end
character, was implemented that way before.
The test is generating a termination character different from the line
end character (to keep it general).
The unpacking of the status word is limited to the 13 bytes that
we want to unpack.

* Removed terminator from status word createn in test fixture

The terminator is already added by expected protocol.

Co-authored-by: Steven Casagrande <scasagrande@galvant.ca>

13420 of 14125 relevant lines covered (95.01%)

0.95 hits per line

Source Files on job 1318.1
  • Tree
  • List 0
  • Changed 1
  • Source Changed 1
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 1191
  • Travis Job 1318.1
  • 0ef885fd on github
  • Prev Job for on master (#1316.1)
  • Next Job for on master (#1322.2)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc