mirror/cypress
1
0
Fork 0
mirror of https://github.com/cypress-io/cypress.git synced 2026-02-05 06:40:56 -06:00
Code Issues Packages Projects Releases Wiki Activity

server: update cache project insertion

Browse Source
This commit is contained in:
Chris Breiding
2017-06-12 11:17:12 -04:00
parent 899a37273f
commit 7b98716d5e
2 changed files with 38 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
packages/server/lib/cache.coffee
View File

@@ -84,10 +84,13 @@ module.exports = {
insertProject: (path) ->
fileUtil.transaction (tx) =>
@_getProjects(tx).then (projects) =>
## bail if we already have this path
return projects if path in projects
## projects are sorted by most recently used, so add a project to
## the start or move it to the start if it already exists
existingIndex = _.findIndex projects, (project) -> project is path
if existingIndex > -1
projects.splice(existingIndex, 1)
projects.push(path)
projects.unshift(path)
tx.set("PROJECTS", projects)
getUser: ->

47
packages/server/test/unit/cache_spec.coffee
View File

@@ -56,27 +56,44 @@ describe "lib/cache", ->
context "projects", ->
describe "#insertProject", ->
it "inserts project by path", ->
cache.insertProject("foo/bar").then =>
cache.__get("PROJECTS").then (projects) ->
expect(projects).to.deep.eq ["foo/bar"]
it "is a noop if project already exists by id", ->
cache.insertProject("foo/bar")
.then =>
cache.insertProject("foo/bar")
.then =>
cache.__get("PROJECTS").then (projects) ->
expect(projects).to.deep.eq(["foo/bar"])
.then ->
cache.__get("PROJECTS")
.then (projects) ->
expect(projects).to.deep.eq ["foo/bar"]
it "can insert multiple projects", ->
it "inserts project at the start", ->
cache.insertProject("foo")
.then ->
cache.insertProject("bar")
.then ->
cache.__get("PROJECTS")
.then (projects) ->
expect(projects).to.deep.eq ["bar", "foo"]
it "can insert multiple projects in a row", ->
Promise.all([
cache.insertProject("baz")
cache.insertProject("bar")
cache.insertProject("foo")
])
.then ->
cache.__get("PROJECTS")
.then (projects) ->
expect(projects).to.deep.eq(["foo", "bar", "baz"])
it "moves project to start if it already exists", ->
Promise.all([
cache.insertProject("foo")
cache.insertProject("bar")
cache.insertProject("baz")
])
.then =>
cache.__get("PROJECTS").then (projects) ->
expect(projects).to.deep.eq(["foo", "bar", "baz"])
.then ->
cache.insertProject("bar")
.then ->
cache.__get("PROJECTS")
.then (projects) ->
expect(projects).to.deep.eq ["bar", "baz", "foo"]
describe "#removeProject", ->
it "removes project by path", ->
@@ -100,7 +117,7 @@ describe "lib/cache", ->
cache.insertProject("/Users/sam/app2")
.then =>
cache.getProjectPaths().then (paths) ->
expect(paths).to.deep.eq ["/Users/brian/app", "/Users/sam/app2"]
expect(paths).to.deep.eq ["/Users/sam/app2", "/Users/brian/app"]
it "removes any paths which no longer exist on the filesystem", ->
@statAsync.withArgs("/Users/brian/app").resolves()