diff --git a/assets/scripts/app/auth.coffee b/assets/scripts/app/auth.coffee
index b025f4fa..472df011 100644
--- a/assets/scripts/app/auth.coffee
+++ b/assets/scripts/app/auth.coffee
@@ -20,13 +20,16 @@
Travis.__container__.lookup('controller:currentUser').set('content', null)
Travis.__container__.lookup('router:main').send('afterSignOut')
- signIn: ->
- @set('state', 'signing-in')
- url = "#{@endpoint}/auth/post_message?origin=#{@receivingEnd}"
- $('').hide().appendTo('body').attr('src', url)
+ signIn: (data) ->
+ if data
+ @autoSignIn(data)
+ else
+ @set('state', 'signing-in')
+ url = "#{@endpoint}/auth/post_message?origin=#{@receivingEnd}"
+ $('').hide().appendTo('body').attr('src', url)
- autoSignIn: ->
- data = @userDataFrom(Travis.sessionStorage) || @userDataFrom(Travis.storage)
+ autoSignIn: (data) ->
+ data ||= @userDataFrom(Travis.sessionStorage) || @userDataFrom(Travis.storage)
@setData(data) if data
userDataFrom: (storage) ->
diff --git a/assets/scripts/app/controllers/repos.coffee b/assets/scripts/app/controllers/repos.coffee
index e03d98e0..6e47eb73 100644
--- a/assets/scripts/app/controllers/repos.coffee
+++ b/assets/scripts/app/controllers/repos.coffee
@@ -1,7 +1,20 @@
require 'travis/limited_array'
Travis.ReposController = Ember.ArrayController.extend
- defaultTab: 'owned'
+ defaultTab: ( ->
+ if @get('currentUser.id')
+ 'owned'
+ else
+ 'recent'
+ ).property('currentUser')
+
+ currentUserIdDidChange: (->
+ if @get('currentUser.id')
+ @activate('owned')
+ else if @get('tab') == 'owned'
+ @activate('recent')
+ ).observes('currentUser.id')
+
isLoadedBinding: 'content.isLoaded'
needs: ['currentUser', 'repo']
currentUserBinding: 'controllers.currentUser'
@@ -34,7 +47,7 @@ Travis.ReposController = Ember.ArrayController.extend
Ember.run.later(@updateTimes.bind(this), Travis.INTERVALS.updateTimes)
activate: (tab, params) ->
- tab ||= @defaultTab
+ tab ||= @get('defaultTab')
@set('tab', tab)
this["view#{$.camelize(tab)}"](params)
diff --git a/assets/scripts/app/views/repo/show.coffee b/assets/scripts/app/views/repo/show.coffee
index b0eec1a6..3b22fb86 100644
--- a/assets/scripts/app/views/repo/show.coffee
+++ b/assets/scripts/app/views/repo/show.coffee
@@ -10,6 +10,20 @@ Travis.reopen
@get('repos.isLoaded') && @get('repos.length') == 0
).property('repos.isLoaded', 'repos.length')
+ repoIsLoadedDidChange: (->
+ # Ember does not automatically rerender outlets and sometimes 'pane' outlet
+ # in repos/show.hbs is empty when view is rerendered without routing
+ # taking place. Try to render the default outlet in such case
+ # TODO: look into fixing it in more general way
+ pane = Ember.get('_outlets.pane')
+ if @get('controller.repo.isLoaded') && @state == 'inDOM' &&
+ @get('controller.tab') == 'current' && (!pane || pane.state == 'destroyed')
+ view = @get('controller.container').lookup('view:build')
+ view.set('controller', @get('controller.container').lookup('controller:build'))
+ Ember.run.next =>
+ @connectOutlet('pane', view)
+ ).observes('controller.repo.isLoaded')
+
ReposEmptyView: Travis.View.extend
template: ''
diff --git a/assets/scripts/spec/my_repos_spec.coffee b/assets/scripts/spec/my_repos_spec.coffee
new file mode 100644
index 00000000..21afcd25
--- /dev/null
+++ b/assets/scripts/spec/my_repos_spec.coffee
@@ -0,0 +1,52 @@
+describe 'my repos tab', ->
+ describe 'when user is signed in', ->
+ beforeEach ->
+ app '/', user: true
+ waitFor myReposRendered, 'my repositories'
+ runs ->
+ waitFor buildRendered, 'build view'
+
+ it 'is active', ->
+ listsRepos [
+ { slug: 'travis-ci/travis-core', build: { number: 1, url: '/travis-ci/travis-core/builds/1', duration: '30 sec', finishedAt: '3 minutes ago' } }
+ { slug: 'travis-ci/travis-hub', build: { number: 4, url: '/travis-ci/travis-hub/builds/4', duration: '1 min', finishedAt: '-' } }
+ ]
+
+ displaysRepository
+ href: 'http://github.com/travis-ci/travis-core'
+
+ displaysSummary
+ type: 'build'
+ id: 1
+ repo: 'travis-ci/travis-core'
+ commit: '1234567'
+ branch: 'master'
+ compare: '0123456..1234567'
+ finishedAt: '3 minutes ago'
+ duration: '30 sec'
+ message: 'commit message 1'
+
+ describe 'when user is not signed in', ->
+ beforeEach ->
+ app '/'
+ waitFor reposRendered, 'repositories list'
+ runs ->
+ waitFor buildRendered, 'build view'
+
+ it 'is activated when user signs in', ->
+ listsRepos [
+ { slug: 'travis-ci/travis-hub', build: { number: 4, url: '/travis-ci/travis-hub/builds/4', duration: '1 min', finishedAt: '-' } }
+ { slug: 'travis-ci/travis-core', build: { number: 1, url: '/travis-ci/travis-core/builds/1', duration: '30 sec', finishedAt: '3 minutes ago' } }
+ { slug: 'travis-ci/travis-assets', build: { number: 3, url: '/travis-ci/travis-assets/builds/3', duration: '30 sec', finishedAt: 'a day ago' } }
+ ]
+
+ signInUser()
+
+ waitFor myReposRendered, 'my repositories'
+ runs ->
+ waitFor buildRendered, 'build view'
+ runs ->
+ listsRepos [
+ { slug: 'travis-ci/travis-core', build: { number: 1, url: '/travis-ci/travis-core/builds/1', duration: '30 sec', finishedAt: '3 minutes ago' } }
+ { slug: 'travis-ci/travis-hub', build: { number: 4, url: '/travis-ci/travis-hub/builds/4', duration: '1 min', finishedAt: '-' } }
+ ]
diff --git a/assets/scripts/spec/spec_helper.coffee b/assets/scripts/spec/spec_helper.coffee
index 88070108..5714a045 100644
--- a/assets/scripts/spec/spec_helper.coffee
+++ b/assets/scripts/spec/spec_helper.coffee
@@ -1,12 +1,26 @@
minispade.require 'app'
-@app = (url) ->
+@signInUser = ->
+ # for now let's just use harcoded data to log in the user,
+ # we may extend it in the future to pass specific user data
+ Travis.auth.signIn
+ user:
+ id: 1
+ email: 'tyrion@example.org'
+ login: 'tyrion'
+ token: 'abcdef'
+ token: 'abcdef'
+
+@app = (url, options = {}) ->
# TODO: this should wait till app is initialized, not some
# arbitrary amount of time
waits(50)
runs ->
Travis.reset()
Travis.auth.signOut()
+
+ if options.user
+ signInUser()
url = "/#{url}" unless url.match /^\//
Travis.__container__.lookup('router:main').handleURL(url)
diff --git a/assets/scripts/spec/support/conditions.coffee b/assets/scripts/spec/support/conditions.coffee
index 3a2250a8..66188ae2 100644
--- a/assets/scripts/spec/support/conditions.coffee
+++ b/assets/scripts/spec/support/conditions.coffee
@@ -5,6 +5,7 @@
-> $(selector).text().trim() == text
@reposRendered = notEmpty('#repos li.selected')
+@myReposRendered = -> notEmpty('#repos li.selected')() && $('#left #tab_owned').hasClass('active')
@buildRendered = notEmpty('#summary .number')
@buildsRendered = notEmpty('#builds .number')
@jobRendered = notEmpty('#summary .number')
diff --git a/assets/scripts/spec/support/mocks.coffee b/assets/scripts/spec/support/mocks.coffee
index ea256dab..d74bd61f 100644
--- a/assets/scripts/spec/support/mocks.coffee
+++ b/assets/scripts/spec/support/mocks.coffee
@@ -6,6 +6,14 @@ repos = [
{ id: '3', owner: 'travis-ci', name: 'travis-hub', slug: 'travis-ci/travis-hub', build_ids: [4], last_build_id: 4, last_build_number: 4, last_build_result: undefined, last_build_duration: undefined, last_build_started_at: '2012-07-02T00:02:00Z', last_build_finished_at: undefined, description: 'Description of travis-hub'},
]
+reposByName = (name) ->
+ # this is hardcoded as well as user is hardcoded in app() helper,
+ # please make it more flexible if needed
+ if name == 'tyrion'
+ [repos[0], repos[2]]
+ else
+ []
+
builds = [
{ id: '1', repository_id: '1', commit_id: 1, job_ids: [1, 2, 3], number: 1, pull_request: false, config: { rvm: ['rbx', '1.9.3', 'jruby'] }, duration: 30, started_at: '2012-07-02T00:00:00Z', finished_at: '2012-07-02T00:00:30Z', state: 'passed' },
{ id: '2', repository_id: '1', commit_id: 2, job_ids: [4], number: 2, pull_request: false, config: { rvm: ['rbx'] } },
@@ -70,8 +78,11 @@ $.mockjax
this.responseText = { repos: [$.detect(repos, (repository) -> repository.slug == slug)] }
else if search = settings.data.search
this.responseText = { repos: $.select(repos, (repository) -> repository.slug.indexOf(search) > -1).toArray() }
+ else if settings.data.member
+ this.responseText = { repos: reposByName(settings.data.member) }
else
- raise "don't know this ditty"
+ console.log settings.data
+ throw 'unknown params for repos'
for repository in repos
$.mockjax
diff --git a/assets/styles/left.sass b/assets/styles/left.sass
index 705bb384..c9a6c515 100644
--- a/assets/styles/left.sass
+++ b/assets/styles/left.sass
@@ -17,14 +17,6 @@
@include border-radius(4px)
background: $color-bg-input inline-image('ui/search.png') no-repeat right 8px
- .tabs
- #tab_owned
- display: none
-
- .tabs
- #tab_owned.display
- display: inline-block
-
.toggle-info
margin: 8px 20px 0 20px
display: inline-block
diff --git a/assets/styles/tabs.sass b/assets/styles/tabs.sass
index 19baa134..01b6c50b 100644
--- a/assets/styles/tabs.sass
+++ b/assets/styles/tabs.sass
@@ -34,14 +34,19 @@
#left
.tabs
+ padding-left: 20px
margin-top: -29px
- li:first-child
- margin-left: 20px
-
#tab_search:not(.active)
display: none
+ #tab_owned
+ display: none
+ #tab_owned.display
+ display: inline-block
+
+
+
#main
.tabs
margin-top: 35px