Refactor assertions to get more useful error messages

.travis.yml

@@ -0,0 +1,37 @@
+language: erlang
+
+env:
+  global:
+    - LUAROCKS_BASE=luarocks-2.0.9
+  matrix:
+    - LUA=lua5.1 LUA_DEV=liblua5.1-dev LUA_VER=5.1 LUA_SFX=5.1 LUA_INCDIR=/usr/include/lua5.1
+    - LUA=lua5.2 LUA_DEV=liblua5.2-dev LUA_VER=5.2 LUA_SFX=5.2 LUA_INCDIR=/usr/include/lua5.2
+    - LUA=luajit LUA_DEV=libluajit-5.1-dev LUA_VER=5.1 LUA_SFX=jit LUA_INCDIR=/usr/include/luajit-2.0
+
+before_install:
+  - if [ $LUA = "luajit" ]; then
+      sudo add-apt-repository ppa:mwild1/ppa -y && sudo apt-get update -y;
+    fi
+  - sudo apt-get install $LUA
+  - sudo apt-get install $LUA_DEV
+  - lua$LUA_SFX -v
+  # Install a recent luarocks release
+  - wget http://luarocks.org/releases/$LUAROCKS_BASE.tar.gz
+  - tar zxvpf $LUAROCKS_BASE.tar.gz
+  - cd $LUAROCKS_BASE
+  - ./configure
+    --lua-version=$LUA_VER --lua-suffix=$LUA_SFX --with-lua-include="$LUA_INCDIR"
+  - sudo make
+  - sudo make install
+  - cd $TRAVIS_BUILD_DIR
+
+install:
+  - sudo -E luarocks install busted
+
+script:
+  - sudo -E busted -v
+
+notifications:
+  email:
+    on_success: change
+    on_failure: always

README.md

@@ -1,12 +1,14 @@
-h1. semver.lua 
+# semver.lua
+
+[![Build Status](https://travis-ci.org/kikito/semver.lua.svg?branch=master)](https://travis-ci.org/kikito/semver.lua)
 
 Semantic versioning for Lua.
 
 See http://semver.org/ for details about semantic versioning.
 
-h1. Documentation
+# Documentation
 
-<pre class="lua">
+``` lua
 local v = require 'semver'
 
 -- two ways of creating it: with separate parameters, or with one string
@@ -52,24 +54,25 @@ v(1,0,0):nextPatch() -- v1.0.1
 v(1,2,3):nextMinor() -- v1.3.0 . Notice the patch resets to 0
 v(1,2,3):nextMajor() -- v2.0.0 . Minor and patch are reset to 0
 
-</pre>
+```
 
+# Installation
 
-h1. Installation
+Just copy the semver.lua file wherever you want it (for example on a lib/ folder). Then write this in any Lua file where you want to use it. You must assign the require to a global or local variable (I use a local `v`):
 
-Just copy the semver.lua file wherever you want it (for example on a lib/ folder). Then write this in any Lua file where you want to use it. You must assign the require to a global or local variable (I use a local @v@):
+``` lua
+local v = require 'semver'
+```
 
-<pre>local v = require 'semver'</pre>
+Using `v` allows for the nice string syntax: `v'1.2.3-alpha'`.
 
-Using @v@ allows for the nice string syntax: @v'1.2.3-alpha'@.
+The `package.path` variable must be configured so that the folder in which middleclass.lua is copied is available, of course.
-
-The @package.path@ variable must be configured so that the folder in which middleclass.lua is copied is available, of course.
 
 Please make sure that you read the license, too (for your convenience it's now included at the beginning of the semver.lua file).
 
-h1. Notes about version comparison
+# Notes about version comparison
 
-Version comparison is done according to the specs:
+Version comparison is done according to the semver 2.0.0 specs:
 
 Major, minor, and patch versions are always compared numerically.
 
@@ -79,11 +82,18 @@ Pre-release and build version precedence MUST be determined by comparing each do
 * Identifiers with letters or dashes are compared lexically in ASCII sort order.
 * Numeric identifiers always have lower precedence than non-numeric identifiers
 
-h1. Specs
+# Specs
 
-This project uses "telescope":https://github.com/norman/telescope for its specs. If you want to run the specs, you will have to install telescope first. Then just execute the following from the root inspect folder:
+This project uses "busted":http://olivinelabs.com/busted/ for its specs. If you want to run the specs, you will have to install telescope first. Then just execute the following from the root inspect folder:
 
-<pre>
+```
-tsc -f spec/*
+busted
-</pre>
+```
+
+# Changelog
+
+## v1.1.1:
+* Removed global variable which was declared by mistake
+* Changed spec tool from telescope to busted
+* Changed README format from textile to markdown
 

rockspecs/semver-1.1.0-1.rockspec

@@ -0,0 +1,25 @@
+package = "semver"
+version = "1.1.0-1"
+source = {
+   url = "git://github.com/kikito/semver.lua.git",
+}
+description = {
+   summary = "An implementation of semantic versioning (semver.org) in Lua",
+   detailed = [[
+      See details in http://semver.org
+   ]],
+   license = "MIT",
+   homepage = "https://github.com/kikito/semver.lua"
+}
+dependencies = {
+   "lua >= 5.1"
+}
+
+build = {
+  type = "none",
+  install = {
+    lua = {
+      "semver.lua"
+    },
+  }
+}

rockspecs/semver-1.1.1-1.rockspec

@@ -0,0 +1,25 @@
+package = "semver"
+version = "1.1.1-1"
+source = {
+   url = "git://github.com/kikito/semver.lua.git",
+}
+description = {
+   summary = "An implementation of semantic versioning (semver.org 2.0.0) in Lua",
+   detailed = [[
+      See details in http://semver.org
+   ]],
+   license = "MIT",
+   homepage = "https://github.com/kikito/semver.lua"
+}
+dependencies = {
+   "lua >= 5.1"
+}
+
+build = {
+  type = "none",
+  install = {
+    lua = {
+      "semver.lua"
+    },
+  }
+}

semver.lua

@@ -1,10 +1,32 @@
--- semver.lua - v1.0 (2012-01)
+local semver = {
--- Copyright (c) 2012 Enrique García Cota
+  _VERSION     = '1.1.1',
--- Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+  _DESCRIPTION = 'semver for Lua',
--- The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
+  _URL         = 'https://github.com/kikito/semver.lua',
--- THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+  _LICENSE     = [[
--- See http://semver.org for details
+    MIT LICENSE
-local version
+
+    Copyright (c) 2015 Enrique García Cota
+
+    Permission is hereby granted, free of charge, to any person obtaining a
+    copy of tother software and associated documentation files (the
+    "Software"), to deal in the Software without restriction, including
+    without limitation the rights to use, copy, modify, merge, publish,
+    distribute, sublicense, and/or sell copies of the Software, and to
+    permit persons to whom the Software is furnished to do so, subject to
+    the following conditions:
+
+    The above copyright notice and tother permission notice shall be included
+    in all copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+    OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+    MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+    IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+    CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+    TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+    SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+  ]]
+}
 
 local function checkPositiveInteger(number, name)
   assert(number >= 0, name .. ' must be a valid positive number')
@@ -17,6 +39,7 @@ end
 
 -- splitByDot("a.bbc.d") == {"a", "bbc", "d"}
 local function splitByDot(str)
+  str = str or ""
   local t, count = {}, 0
   str:gsub("([^%.]+)", function(c)
     count = count + 1
@@ -45,7 +68,7 @@ end
 
 local function parseBuild(buildWithSign)
   if buildWithSign then
-    build = buildWithSign:match("^%+(%w[%.%w-]*)$")
+    local build = buildWithSign:match("^%+(%w[%.%w-]*)$")
     assert(build, ("The build %q is not a + sign followed by alphanumerics, dots and slashes"):format(buildWithSign))
     return build
   end
@@ -76,49 +99,79 @@ local function compare(a,b)
   return a == b and 0 or a < b and -1 or 1
 end
 
-local function compareIds(selfId, otherId)
+local function compareNilPrereleases(mine, other)
-  if not selfId and not otherId then return 0
+  if mine == other then return  0
-  elseif not selfId then return             1
+  elseif not mine  then return  1
-  elseif not otherId then return           -1
+  elseif not other then return -1
-  end
+  end -- else return nil
+end
 
+-- notice that builds compare nils inversely than prereleases (the -1 and 1 are switched)
+local function compareNilBuilds(mine, other)
+  if mine == other then return  0
+  elseif not mine  then return -1
+  elseif not other then return  1
+  end -- else return nil
+end
+
+local function compareIds(selfId, otherId)
   local selfNumber, otherNumber = tonumber(selfId), tonumber(otherId)
 
   if selfNumber and otherNumber then -- numerical comparison
     return compare(selfNumber, otherNumber)
-  elseif selfNumber then -- numericals are always smaller than alphanums
+  -- numericals are always smaller than alphanums
+  elseif selfNumber then
     return -1
+  elseif otherNumber then
+    return 1
   else
     return compare(selfId, otherId) -- alphanumerical comparison
   end
 end
 
-local function smallerPrereleaseOrBuild(mine, his)
+local function smallerIds(mine, other, compareNil)
-  if mine == his then return false end
+  local myIds, otherIds = splitByDot(mine), splitByDot(other)
-
-  local myIds, hisIds = splitByDot(mine), splitByDot(his)
   local myLength = #myIds
-  local comparison
+  local comparison, myId, otherId
 
   for i = 1, myLength do
-    comparison = compareIds(myIds[i], hisIds[i])
+    myId, otherId = myIds[i], otherIds[i]
-    if comparison ~= 0 then return comparison == -1 end
+    comparison = compareNil(myId, otherId) or compareIds(myId, otherId)
+    if comparison ~= 0 then
+      return comparison == -1
+    end
     -- if comparison == 0, continue loop
   end
 
-  return myLength < #hisIds
+  return myLength < #otherIds
+end
+
+local function smallerPrerelease(mine, other)
+  if mine == other  or not mine then return false
+  elseif not other then return true
+  end
+
+  return smallerIds(mine, other, compareNilPrereleases)
+end
+
+local function smallerBuild(mine, other)
+  if mine == other or not other then return false
+  elseif not mine then return true
+  end
+
+  return smallerIds(mine, other, compareNilBuilds)
 end
 
 local methods = {}
 
 function methods:nextMajor()
-  return version(self.major + 1, 0, 0)
+  return semver(self.major + 1, 0, 0)
 end
 function methods:nextMinor()
-  return version(self.major, self.minor + 1, 0)
+  return semver(self.major, self.minor + 1, 0)
 end
 function methods:nextPatch()
-  return version(self.major, self.minor, self.patch + 1)
+  return semver(self.major, self.minor, self.patch + 1)
 end
 
 local mt = { __index = methods }
@@ -133,10 +186,8 @@ function mt:__lt(other)
   return self.major < other.major or
          self.minor < other.minor or
          self.patch < other.patch or
-         (self.prerelease and not other.prerelease) or
+         smallerPrerelease(self.prerelease, other.prerelease) or
-         smallerPrereleaseOrBuild(self.prerelease, other.prerelease) or
+         smallerBuild(self.build, other.build)
-         (not self.build and other.build) or
-         smallerPrereleaseOrBuild(self.build, other.build)
 end
 function mt:__pow(other)
   return self.major == other.major and
@@ -149,9 +200,7 @@ function mt:__tostring()
   return table.concat(buffer)
 end
 
-
+local function new(major, minor, patch, prerelease, build)
--- defined as local at the begining of the file
-version = function(major, minor, patch, prerelease, build)
   assert(major, "At least one parameter is needed")
 
   if type(major) == 'string' then
@@ -168,4 +217,7 @@ version = function(major, minor, patch, prerelease, build)
   return setmetatable(result, mt)
 end
 
-return version
+setmetatable(semver, { __call = function(_, ...) return new(...) end })
+semver._VERSION= semver(semver._VERSION)
+
+return semver

spec/custom_assertions.lua

@@ -0,0 +1,21 @@
+local assert = require("luassert")
+local say    = require("say")
+
+local function less(state, arguments)
+  return arguments[1] < arguments[2]
+end
+
+local function greater(state, arguments)
+  return arguments[1] > arguments[2]
+end
+
+say:set_namespace("en")
+say:set("assertion.less.positive", "Expected %s to be smaller than %s")
+say:set("assertion.less.negative", "Expected %s to not be smaller than %s")
+assert:register("assertion", "less", less, "assertion.less.positive", "assertion.less.negative")
+
+say:set("assertion.greater.positive", "Expected %s to be greater than %s")
+say:set("assertion.greater.negative", "Expected %s to not be greater than %s")
+assert:register("assertion", "greater", greater, "assertion.greater.positive", "assertion.greater.negative")
+
+

spec/semver_spec.lua

@@ -1,18 +1,20 @@
+require 'spec.custom_assertions'
+
 local v = require 'semver'
 
 local function checkVersion(ver, major, minor, patch, prerelease, build)
-  assert_equal(major, ver.major)
+  assert.equal(major, ver.major)
-  assert_equal(minor, ver.minor)
+  assert.equal(minor, ver.minor)
-  assert_equal(patch, ver.patch)
+  assert.equal(patch, ver.patch)
-  assert_equal(prerelease, ver.prerelease)
+  assert.equal(prerelease, ver.prerelease)
-  assert_equal(build, ver.build)
+  assert.equal(build, ver.build)
 end
 
-context('semver', function()
+describe('semver', function()
 
-  context('creation', function()
+  describe('creation', function()
 
-    context('from numbers', function()
+    describe('from numbers', function()
       it('parses 3 numbers correctly', function()
         checkVersion(v(1,2,3), 1,2,3)
       end)
@@ -62,170 +64,193 @@ context('semver', function()
 
     describe('errors', function()
       test('no parameters are passed', function()
-        assert_error(function() v() end)
+        assert.error(function() v() end)
       end)
       test('negative numbers', function()
-        assert_error(function() v(-1, 0, 0) end)
+        assert.error(function() v(-1, 0, 0) end)
-        assert_error(function() v( 0,-1, 0) end)
+        assert.error(function() v( 0,-1, 0) end)
-        assert_error(function() v( 0, 0,-1) end)
+        assert.error(function() v( 0, 0,-1) end)
       end)
       test('floats', function()
-        assert_error(function() v(.1, 0, 0) end)
+        assert.error(function() v(.1, 0, 0) end)
-        assert_error(function() v( 0,.1, 0) end)
+        assert.error(function() v( 0,.1, 0) end)
-        assert_error(function() v( 0, 0,.1) end)
+        assert.error(function() v( 0, 0,.1) end)
       end)
       test('empty string', function()
-        assert_error(function() v("") end)
+        assert.error(function() v("") end)
       end)
       test('garbage at the beginning of the string', function()
-        assert_error(function() v("foobar1.2.3") end)
+        assert.error(function() v("foobar1.2.3") end)
       end)
       test('garbage at the end of the string', function()
-        assert_error(function() v("1.2.3foobar") end)
+        assert.error(function() v("1.2.3foobar") end)
       end)
       test('a non-string or number is passed', function()
-        assert_error(function() v({}) end)
+        assert.error(function() v({}) end)
       end)
       test('an invalid prerelease', function()
-        assert_error(function() v'1.2.3-%?' end)
+        assert.error(function() v'1.2.3-%?' end)
       end)
       test('an invalid build', function()
-        assert_error(function() v'1.2.3+%?' end)
+        assert.error(function() v'1.2.3+%?' end)
       end)
     end)
   end)
 
   describe("tostring", function()
     it("works with major, minor and patch", function()
-      assert_equal("1.2.3", tostring(v(1,2,3)))
+      assert.equal("1.2.3", tostring(v(1,2,3)))
     end)
 
     it("works with a prerelease", function()
-      assert_equal("1.2.3-beta", tostring(v(1,2,3,'beta')))
+      assert.equal("1.2.3-beta", tostring(v(1,2,3,'beta')))
     end)
     it("works with a build", function()
-      assert_equal("1.2.3+foobar", tostring(v(1,2,3,nil,'foobar')))
+      assert.equal("1.2.3+foobar", tostring(v(1,2,3,nil,'foobar')))
     end)
     it("works with a prerelease and a build", function()
-      assert_equal("1.2.3-alpha+foobar", tostring(v'1.2.3-alpha+foobar'))
+      assert.equal("1.2.3-alpha+foobar", tostring(v'1.2.3-alpha+foobar'))
     end)
   end)
 
   describe("==", function()
     it("is true when major, minor and patch are the same", function()
-      assert_equal(v(1,2,3), v'1.2.3')
+      assert.equal(v(1,2,3), v'1.2.3')
     end)
     it("is false when major, minor and patch are not the same", function()
-      assert_not_equal(v(1,2,3), v(4,5,6))
+      assert.not_equal(v(1,2,3), v(4,5,6))
     end)
     it("false if all is the same except the prerelease", function()
-      assert_not_equal(v(1,2,3), v'1.2.3-alpha')
+      assert.not_equal(v(1,2,3), v'1.2.3-alpha')
     end)
     it("false if all is the same except the build", function()
-      assert_not_equal(v(1,2,3), v'1.2.3+peter.1')
+      assert.not_equal(v(1,2,3), v'1.2.3+peter.1')
     end)
   end)
 
   describe("<", function()
     test("true if major < minor", function()
-      assert_less_than(v'1.100.10', v'2.0.0')
+      assert.less(v'1.100.10', v'2.0.0')
     end)
     test("false if major > minor", function()
-      assert_greater_than(v'2', v'1')
+      assert.greater(v'2', v'1')
     end)
     test("true if major = major but minor < minor", function()
-      assert_less_than(v'1.2.0', v'1.3.0')
+      assert.less(v'1.2.0', v'1.3.0')
     end)
     test("false if minor < minor", function()
-      assert_greater_than(v'1.1', v'1.0')
+      assert.greater(v'1.1', v'1.0')
     end)
     test("true if major =, minor =, but patch <", function()
-      assert_less_than(v'0.0.1', v'0.0.10')
+      assert.less(v'0.0.1', v'0.0.10')
     end)
     test("false if major =, minor =, but patch >", function()
-      assert_greater_than(v'0.0.2', v'0.0.1')
+      assert.greater(v'0.0.2', v'0.0.1')
     end)
     describe("prereleases", function()
       test("false if exact same prerelease", function()
-        assert_false(v'1.0.0-beta' < v'1.0.0-beta')
+        assert.not_less(v'1.0.0-beta', v'1.0.0-beta')
       end)
-      test("a prerelease version is less than the official version", function()
+      test("#focus a prerelease version is less than the official version", function()
-        assert_less_than(v'1.0.0-rc1', v'1.0.0')
+        assert.less(v'1.0.0-rc1', v'1.0.0')
+        assert.not_greater(v'1.0.0-rc1', v'1.0.0')
+        assert.greater(v'1.2.3', v'1.2.3-alpha')
+        assert.not_less(v'1.2.3', v'1.2.3-alpha')
       end)
       test("identifiers with only digits are compared numerically", function()
-        assert_less_than(v'1.0.0-1', v'1.0.0-2')
+        assert.less(v'1.0.0-1', v'1.0.0-2')
-        assert_less_than(v'1.0.0-2', v'1.0.0-10')
+        assert.not_greater(v'1.0.0-1', v'1.0.0-2')
       end)
-      test("itendifiers with letters or dashes are compared lexiconumerically", function()
+      test("idendifiers with letters or dashes are compared lexiconumerically", function()
-        assert_less_than(v'1.0.0-alpha', v'1.0.0-beta')
+        assert.less(v'1.0.0-alpha', v'1.0.0-beta')
-        assert_less_than(v'1.0.0-alpha-10', v'1.0.0-alpha-2')
+        assert.less(v'1.0.0-alpha-10', v'1.0.0-alpha-2')
+        assert.not_greater(v'1.0.0-alpha', v'1.0.0-beta')
+        assert.not_greater(v'1.0.0-alpha-10', v'1.0.0-alpha-2')
       end)
       test("numerical ids always have less priority than lexiconumericals", function()
-        assert_less_than(v'1.0.0-1', v'1.0.0-alpha')
+        assert.less(v'1.0.0-1', v'1.0.0-alpha')
-        assert_less_than(v'1.0.0-2', v'1.0.0-1asdf')
+        assert.less(v'1.0.0-2', v'1.0.0-1asdf')
+        assert.not_greater(v'1.0.0-1', v'1.0.0-alpha')
+        assert.not_greater(v'1.0.0-2', v'1.0.0-1asdf')
       end)
       test("identifiers can be separated by colons; they must be compared individually", function()
-        assert_less_than(v'1.0.0-alpha',   v'1.0.0-alpha.1')
+        --assert.less(v'1.0.0-alpha'   , v'1.0.0-alpha.1')
-        assert_less_than(v'1.0.0-alpha.1', v'1.0.0-beta.2')
+        --assert.less(v'1.0.0-alpha.1' , v'1.0.0-beta.2')
-        assert_less_than(v'1.0.0-beta.2',  v'1.0.0-beta.11')
+        --assert.less(v'1.0.0-beta.2'  , v'1.0.0-beta.11')
-        assert_less_than(v'1.0.0-beta.11', v'1.0.0-rc.1')
+        --assert.less(v'1.0.0-beta.11' , v'1.0.0-rc.1')
+        assert.not_greater(v'1.0.0-alpha', v'1.0.0-alpha.1')
+        --assert.not_greater(v'1.0.0-alpha.1', v'1.0.0-beta.2')
+        --assert.not_greater(v'1.0.0-beta.2' , v'1.0.0-beta.11')
+        --assert.not_greater(v'1.0.0-beta.11', v'1.0.0-rc.1')
       end)
     end)
     describe("builds", function()
       test("false if exact same build", function()
-        assert_false(v'1.0.0+build1' < v'1.0.0+build1')
+        assert.not_less(v'1.0.0+build1', v'1.0.0+build1')
       end)
       test("a regular (not-build) version is always less than a build version", function()
-        assert_less_than(v'1.0.0', v'1.0.0+12')
+        assert.less(v'1.0.0', v'1.0.0+12')
+        assert.less(v'1.0.0', v'1.0.0+12')
       end)
       test("identifiers with only digits are compared numerically", function()
-        assert_less_than(v'1.0.0+1', v'1.0.0+2')
+        assert.less(v'1.0.0+1', v'1.0.0+2')
-        assert_less_than(v'1.0.0+2', v'1.0.0+10')
+        assert.less(v'1.0.0+2', v'1.0.0+10')
+        assert.not_greater(v'1.0.0+1', v'1.0.0+2')
+        assert.not_greater(v'1.0.0+2', v'1.0.0+10')
       end)
       test("idendifiers with letters or dashes are compared lexiconumerically", function()
-        assert_less_than(v'1.0.0+build1',  v'1.0.0+build2')
+        assert.less(v'1.0.0+build1' , v'1.0.0+build2')
-        assert_less_than(v'1.0.0+build10', v'1.0.0+build2')
+        assert.less(v'1.0.0+build10', v'1.0.0+build2')
+        assert.not_greater(v'1.0.0+build1'  , v'1.0.0+build2')
+        assert.not_greater(v'1.0.0+build10' , v'1.0.0+build2')
       end)
       test("numerical ids always have less priority than lexiconumericals", function()
-        assert_less_than(v'1.0.0+1', v'1.0.0+build1')
+        assert.less(v'1.0.0+1', v'1.0.0+build1')
-        assert_less_than(v'1.0.0+2', v'1.0.0+1build')
+        assert.less(v'1.0.0+2', v'1.0.0+1build')
+        assert.not_greater(v'1.0.0+1', v'1.0.0+build1')
+        assert.not_greater(v'1.0.0+2', v'1.0.0+1build')
       end)
       test("identifiers can be separated by colons; they must be compared individually", function()
-        assert_less_than(v'1.0.0+0.3.7', v'1.3.7+build')
+        assert.less(v'1.0.0+0.3.7', v'1.3.7+build')
-        assert_less_than(v'1.3.7+build', v'1.3.7+build.2.b8f12d7')
+        assert.less(v'1.3.7+build', v'1.3.7+build.2.b8f12d7')
-        assert_less_than(v'1.3.7+build.2.b8f12d7', v'1.3.7+build.11.e0f985a')
+        assert.less(v'1.3.7+build.2.b8f12d7', v'1.3.7+build.11.e0f985a')
+        assert.not_greater(v'1.0.0+0.3.7', v'1.3.7+build')
+        assert.not_greater(v'1.3.7+build', v'1.3.7+build.2.b8f12d7')
+        assert.not_greater(v'1.3.7+build.2.b8f12d7', v'1.3.7+build.11.e0f985a')
       end)
     end)
-    test("prereleases + builds", function()
+    test("#focus prereleases + builds", function()
-      assert_less_than(v'1.0.0-rc.1', v'1.0.0-rc.1+build.1')
+      --assert.less(v'1.0.0-rc.1', v'1.0.0-rc.1+build.1')
-      assert_less_than(v'1.0.0-rc.1+build.1', v'1.0.0')
+      --assert.less(v'1.0.0-rc.1+build.1', v'1.0.0')
+      --assert.not_greater(v'1.0.0-rc.1', v'1.0.0-rc.1+build.1')
+      assert.not_greater(v'1.0.0-rc.1+build.1', v'1.0.0')
     end)
   end)
 
   describe("nextPatch", function()
     it("increases the patch number by 1", function()
-      assert_equal(v'1.0.1', v'1.0.0':nextPatch())
+      assert.equal(v'1.0.1', v'1.0.0':nextPatch())
     end)
     it("resets prerelease and build", function()
-      assert_equal(v'1.0.1', v'1.0.0-a+b':nextPatch())
+      assert.equal(v'1.0.1', v'1.0.0-a+b':nextPatch())
     end)
   end)
 
   describe("nextMinor", function()
     it("increases the minor number by 1", function()
-      assert_equal(v'1.2.0', v'1.1.0':nextMinor())
+      assert.equal(v'1.2.0', v'1.1.0':nextMinor())
     end)
     it("resets the patch number, prerelease and build", function()
-      assert_equal(v'1.2.0', v'1.1.7-a+b':nextMinor())
+      assert.equal(v'1.2.0', v'1.1.7-a+b':nextMinor())
     end)
   end)
 
   describe("nextMajor", function()
     it("increases the major number by 1", function()
-      assert_equal(v'2.0.0', v'1.0.0':nextMajor())
+      assert.equal(v'2.0.0', v'1.0.0':nextMajor())
     end)
     it("resets the minor, patch, prerelease and build", function()
-      assert_equal(v'2.0.0', v'1.2.3-a+b':nextMajor())
+      assert.equal(v'2.0.0', v'1.2.3-a+b':nextMajor())
     end)
   end)
 
@@ -235,25 +260,32 @@ context('semver', function()
   -- in other words, "it's safe to upgrade from a to b"
   describe("^", function()
     test("true for self", function()
-      assert_true(v(1,2,3) ^ v(1,2,3))
+      assert.is_true(v(1,2,3) ^ v(1,2,3))
     end)
     test("different major versions mean it's always unsafe", function()
-      assert_false(v(2,0,0) ^ v(3,0,0))
+      assert.is_false(v(2,0,0) ^ v(3,0,0))
-      assert_false(v(2,0,0) ^ v(1,0,0))
+      assert.is_false(v(2,0,0) ^ v(1,0,0))
     end)
 
     test("patches, prereleases and builds are ignored", function()
-      assert_true(v(1,2,3) ^ v(1,2,0))
+      assert.is_true(v(1,2,3) ^ v(1,2,0))
-      assert_true(v(1,2,3) ^ v(1,2,5))
+      assert.is_true(v(1,2,3) ^ v(1,2,5))
-      assert_true(v(1,2,3,'foo') ^ v(1,2,3))
+      assert.is_true(v(1,2,3,'foo') ^ v(1,2,3))
-      assert_true(v(1,2,3,nil,'bar') ^ v(1,2,3))
+      assert.is_true(v(1,2,3,nil,'bar') ^ v(1,2,3))
     end)
 
     test("it's safe to upgrade to a newer minor version", function()
-      assert_true(v(1,2,0) ^ v(1,5,0))
+      assert.is_true(v(1,2,0) ^ v(1,5,0))
     end)
     test("it's unsafe to downgrade to an earlier minor version", function()
-      assert_false(v(1,5,0) ^ v(1,2,0))
+      assert.is_false(v(1,5,0) ^ v(1,2,0))
+    end)
+  end)
+
+  describe("_VERSION", function()
+    it("can be extracted from the lib", function()
+      local x = v._VERSION
+      assert.equal('table', type(x))
     end)
   end)