@@ -113,12 +113,10 @@ public async Task Path_Is_Initialized()
113113 public async Task Repository_Name_Is_Appended_To_Base_Path ( )
114114 {
115115 var target = CreateTarget ( ) ;
116- var repository = Substitute . For < IRepositoryModel > ( ) ;
117116
118- repository . Name . Returns ( "repo" ) ;
119- SetRepository ( target . GitHubTab , repository ) ;
117+ SetRepository ( target . GitHubTab , CreateRepositoryModel ( "owner" , "repo" ) ) ;
120118
121- Assert . That ( target . Path , Is . EqualTo ( "d:\\ efault\\ path\\ repo" ) ) ;
119+ Assert . That ( target . Path , Is . EqualTo ( "d:\\ efault\\ path\\ owner \\ repo" ) ) ;
122120 }
123121
124122 [ Test ]
@@ -135,10 +133,7 @@ public async Task PathError_Is_Not_Set_When_No_Repository_Selected()
135133 public async Task PathError_Is_Set_For_Existing_Destination ( )
136134 {
137135 var target = CreateTarget ( ) ;
138- var repository = Substitute . For < IRepositoryModel > ( ) ;
139-
140- repository . Name . Returns ( "repo" ) ;
141- SetRepository ( target . GitHubTab , repository ) ;
136+ SetRepository ( target . GitHubTab , CreateRepositoryModel ( "owner" , "repo" ) ) ;
142137 target . Path = "d:\\ exists" ;
143138
144139 Assert . That ( target . PathError , Is . EqualTo ( Resources . DestinationAlreadyExists ) ) ;
@@ -148,13 +143,14 @@ public async Task PathError_Is_Set_For_Existing_Destination()
148143 public async Task Repository_Name_Replaces_Last_Part_Of_Non_Base_Path ( )
149144 {
150145 var target = CreateTarget ( ) ;
151- var repository = Substitute . For < IRepositoryModel > ( ) ;
152146
153- target . Path = "d:\\ efault\\ foo" ;
154- repository . Name . Returns ( "repo" ) ;
155- SetRepository ( target . GitHubTab , repository ) ;
147+ var owner = "owner" ;
148+ target . Path = "d:\\ efault" ;
149+ SetRepository ( target . GitHubTab , CreateRepositoryModel ( owner , "name" ) ) ;
150+ target . Path = $ "d:\\ efault\\ { owner } \\ foo";
151+ SetRepository ( target . GitHubTab , CreateRepositoryModel ( owner , "repo" ) ) ;
156152
157- Assert . That ( target . Path , Is . EqualTo ( "d:\\ efault\\ repo" ) ) ;
153+ Assert . That ( target . Path , Is . EqualTo ( $ "d:\\ efault\\ { owner } \\ repo") ) ;
158154 }
159155
160156 [ Test ]
@@ -174,7 +170,7 @@ public async Task Clone_Is_Enabled_When_Repository_Selected()
174170
175171 await target . InitializeAsync ( null ) ;
176172
177- SetRepository ( target . GitHubTab , Substitute . For < IRepositoryModel > ( ) ) ;
173+ SetRepository ( target . GitHubTab , CreateRepositoryModel ( ) ) ;
178174
179175 Assert . That ( target . Clone . CanExecute ( null ) , Is . True ) ;
180176 }
@@ -186,7 +182,7 @@ public async Task Clone_Is_Disabled_When_Has_PathError()
186182
187183 await target . InitializeAsync ( null ) ;
188184
189- SetRepository ( target . GitHubTab , Substitute . For < IRepositoryModel > ( ) ) ;
185+ SetRepository ( target . GitHubTab , CreateRepositoryModel ( ) ) ;
190186 Assert . That ( target . Clone . CanExecute ( null ) , Is . True ) ;
191187
192188 target . Path = "d:\\ exists" ;
@@ -260,5 +256,13 @@ static RepositoryCloneViewModel CreateTarget(
260256 enterpriseTab ,
261257 urlTab ) ;
262258 }
259+
260+ static IRepositoryModel CreateRepositoryModel ( string owner = "owner" , string name = "repo" )
261+ {
262+ var repository = Substitute . For < IRepositoryModel > ( ) ;
263+ repository . Owner . Returns ( owner ) ;
264+ repository . Name . Returns ( name ) ;
265+ return repository ;
266+ }
263267 }
264268}
0 commit comments