Bersaelor / KDTree

Swift implementation of a k-dimensional binary space partitioning tree.
MIT License
153 stars 26 forks source link

Feature/test matrix on travis #34

Closed Bersaelor closed 6 years ago

chriseidhof commented 6 years ago

If you're not super committed to travis you could also consider using circleci, they even allow you to ssh into the machine for debugging stuff.

Bersaelor commented 6 years ago

Yeah, i've used both so far, but in this case the error messages were all clear. It also doesn't work for me when I run the tests locally via Dockerized Ubuntu, so travis hasn't caused to much problem. Currently it was failing due to

/home/travis/build/Bersaelor/KDTree/Tests/KDTreeTests/KDTreeTests.swift:18:42: error: use of unresolved identifier 'arc4random'
        return (end-start)*CGFloat(Float(arc4random()) / Float(UINT32_MAX)) + start
                                         ^~~~~~~~~~

which of course should be random() on linux.

Bersaelor commented 6 years ago

I also intentionally switched from testing on iPhone 6S iOS 9.3 to iPhone 5, so I get a 32bit device test. Which now fails in interesting ways: https://travis-ci.org/Bersaelor/KDTree/jobs/301375444

(Locally I get Fatal error: Not enough bits to represent a signed value, haven't seen that one before, happen during self[Int(arc4random())%self.count])