Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file | 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file |
| 2 // for details. All rights reserved. Use of this source code is governed by a | 2 // for details. All rights reserved. Use of this source code is governed by a |
| 3 // BSD-style license that can be found in the LICENSE file. | 3 // BSD-style license that can be found in the LICENSE file. |
| 4 | 4 |
| 5 library oauth2_test; | 5 library oauth2_test; |
| 6 | 6 |
| 7 import 'dart:io'; | 7 import 'dart:io'; |
| 8 import 'dart:json'; | 8 import 'dart:json'; |
| 9 import 'dart:uri'; | 9 import 'dart:uri'; |
| 10 | 10 |
| (...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 142 | 142 |
| 143 response.outputStream.close(); | 143 response.outputStream.close(); |
| 144 }); | 144 }); |
| 145 | 145 |
| 146 pub.kill(); | 146 pub.kill(); |
| 147 | 147 |
| 148 credentialsFile(server, 'new access token').scheduleValidate(); | 148 credentialsFile(server, 'new access token').scheduleValidate(); |
| 149 | 149 |
| 150 run(); | 150 run(); |
| 151 }); | 151 }); |
| 152 | |
| 153 test('with server-rejected credentials, authenticates again and saves ' | |
| 154 'credentials.json', () { | |
| 155 var server = new ScheduledServer(); | |
| 156 credentialsFile(server, 'access token').scheduleCreate(); | |
| 157 var pub = startPubLish(server); | |
| 158 | |
| 159 confirmPublish(pub); | |
| 160 | |
| 161 server.handle('GET', '/packages/versions/new.json', (request, response) { | |
| 162 response.statusCode = 401; | |
| 163 response.headers.set('www-authenticate', 'Bearer error="invalid_token",' | |
| 164 ' error_description="your token sucks"'); | |
| 165 response.outputStream.writeString(JSON.stringify({ | |
| 166 'error': {'message': 'your token sucks'} | |
| 167 })); | |
| 168 response.outputStream.close(); | |
| 169 }); | |
| 170 | |
| 171 expectLater(pub.nextErrLine(), equals('OAuth2 authorization failed (your ' | |
| 172 'token sucks).')); | |
|
Bob Nystrom
2012/12/17 18:41:58
If we show the error before retrying, I think we s
nweiz
2012/12/17 21:18:48
The user has to re-authorize before retrying, whic
| |
| 173 // TODO(rnystrom): The confirm line is run together with this one because | |
| 174 // in normal usage, the user will have entered a newline on stdin which | |
| 175 // gets echoed to the terminal. Do something better here? | |
| 176 expectLater(pub.nextLine(), equals( | |
| 177 'Looks great! Are you ready to upload your package (y/n)? ' | |
| 178 'Pub needs your authorization to upload packages on your behalf.')); | |
| 179 pub.kill(); | |
| 180 run(); | |
| 181 }); | |
| 152 } | 182 } |
| 153 | 183 |
| 154 void authorizePub(ScheduledProcess pub, ScheduledServer server, | 184 void authorizePub(ScheduledProcess pub, ScheduledServer server, |
| 155 [String accessToken="access token"]) { | 185 [String accessToken="access token"]) { |
| 156 // TODO(rnystrom): The confirm line is run together with this one because | 186 // TODO(rnystrom): The confirm line is run together with this one because |
| 157 // in normal usage, the user will have entered a newline on stdin which | 187 // in normal usage, the user will have entered a newline on stdin which |
| 158 // gets echoed to the terminal. Do something better here? | 188 // gets echoed to the terminal. Do something better here? |
| 159 expectLater(pub.nextLine(), equals( | 189 expectLater(pub.nextLine(), equals( |
| 160 'Looks great! Are you ready to upload your package (y/n)? ' | 190 'Looks great! Are you ready to upload your package (y/n)? ' |
| 161 'Pub needs your authorization to upload packages on your behalf.')); | 191 'Pub needs your authorization to upload packages on your behalf.')); |
| (...skipping 23 matching lines...) Expand all Loading... | |
| 185 | 215 |
| 186 response.headers.contentType = new ContentType("application", "json"); | 216 response.headers.contentType = new ContentType("application", "json"); |
| 187 response.outputStream.writeString(JSON.stringify({ | 217 response.outputStream.writeString(JSON.stringify({ |
| 188 "access_token": accessToken, | 218 "access_token": accessToken, |
| 189 "token_type": "bearer" | 219 "token_type": "bearer" |
| 190 })); | 220 })); |
| 191 response.outputStream.close(); | 221 response.outputStream.close(); |
| 192 }); | 222 }); |
| 193 }); | 223 }); |
| 194 } | 224 } |
| OLD | NEW |