1
39%
master: 39%

Ran 18 Apr 2018 09:05AM UTC

Files 29

Run time 2s

Badge

Embed ▾

Committed 18 Apr 2018 09:05AM UTC coverage: 88.889% (-0.4%) from 89.259%

Job # 65.1

Build Type

push

jenkins

Committed by

Commit Message

Fix paket.lock file parser (#38)

* Fix paket.lock file parser

Description
===========

The `paket.lock` file parser has issues with counting leading
whitespaces. The value from the old logic `def newLeadingWhitespaces =
(line =~ /\s/).size()` returns the number of all white space characters
in the current line.

The second part:

```groovy
if (newLeadingWhitespaces > currentLeadingWhitespaces) {
  currentIndent++
} else if (newLeadingWhitespaces < currentLeadingWhitespaces) {
  currentIndent--
}
```

didn't take into account when the indention level suddenly jumped from 3
to 1. I also made the method `List<String>
getAllDependencies(List<String> references)` recursive.
I added a new test covering the failing behavior

Changes
=======

![FIX] `paket.lock` file parsing
![IMPROVE] `getAllDependencies` method by making it recursive
![ADD] new test example

* Refactor the lock parser some more and add more test cases

I switched from `if` statements to `switch/case` because of esthetical
reasons. I added some more guards around the switch arms to limit
crashes. We might need to add a proper parser with error results if and
when we want to support multi type lock files.

Run Details

480 of 540 relevant lines covered (88.89%)

0.89 hits per line

wooga / atlas-paket / 65 / 1
39%
master: 39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 65.1

wooga / atlas-paket / 65 / 1 39% master: 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 65.1

wooga / atlas-paket / 65 / 1
39%
master: 39%

README BADGES
x