refactor: move named arguments to front of tests (#52)

Use Dart's new feature "Named args everywhere"
This commit is contained in:
Aleksey Kulikov 2022-05-04 16:55:40 +03:00 committed by GitHub
parent c73f61d102
commit 984b44d992
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 8 deletions

View file

@ -84,6 +84,7 @@ void main() {
});
test(
testOn: '!linux',
'clones repository with provided keypair',
() {
final cloneDir = Directory.systemTemp.createTempSync('clone');
@ -107,7 +108,6 @@ void main() {
cloneDir.deleteSync(recursive: true);
}
},
testOn: '!linux',
);
test('throws when no credentials is provided', () {
@ -190,6 +190,7 @@ void main() {
});
test(
testOn: '!linux',
'clones repository with provided keypair from memory',
() {
final cloneDir = Directory.systemTemp.createTempSync('clone');
@ -217,7 +218,6 @@ void main() {
cloneDir.deleteSync(recursive: true);
}
},
testOn: '!linux',
);
test('throws when provided keypair from memory is incorrect', () {

View file

@ -282,6 +282,7 @@ void main() {
});
test(
tags: 'remote_fetch',
'fetches data',
() {
Remote.setUrl(
@ -309,10 +310,10 @@ void main() {
expect(stats.receivedBytes, 0);
expect(stats.toString(), contains('TransferProgress{'));
},
tags: 'remote_fetch',
);
test(
tags: 'remote_fetch',
'fetches data with proxy set to auto',
() {
Remote.setUrl(
@ -341,10 +342,10 @@ void main() {
expect(stats.receivedBytes, 0);
expect(stats.toString(), contains('TransferProgress{'));
},
tags: 'remote_fetch',
);
test(
tags: 'remote_fetch',
'uses specified proxy for fetch',
() {
Remote.setUrl(
@ -367,7 +368,6 @@ void main() {
throwsA(isA<LibGit2Error>()),
);
},
tags: 'remote_fetch',
);
test('throws when trying to fetch data with invalid url', () {
@ -381,6 +381,7 @@ void main() {
});
test(
tags: 'remote_fetch',
'fetches data with provided transfer progress callback',
() {
Remote.setUrl(
@ -403,10 +404,10 @@ void main() {
expect(stats.indexedDeltas == callbackStats?.indexedDeltas, true);
expect(stats.receivedBytes == callbackStats?.receivedBytes, true);
},
tags: 'remote_fetch',
);
test(
tags: 'remote_fetch',
'fetches data with provided sideband progress callback',
() {
const sidebandMessage = """
@ -427,10 +428,10 @@ Total 69 (delta 0), reused 1 (delta 0), pack-reused 68
remote.fetch(callbacks: Callbacks(sidebandProgress: sideband));
expect(sidebandOutput.toString(), sidebandMessage);
},
tags: 'remote_fetch',
);
test(
tags: 'remote_fetch',
'fetches data with provided update tips callback',
() {
Remote.setUrl(
@ -469,7 +470,6 @@ Total 69 (delta 0), reused 1 (delta 0), pack-reused 68
remote.fetch(callbacks: Callbacks(updateTips: updateTips));
expect(updateTipsOutput, tipsExpected);
},
tags: 'remote_fetch',
);
test('pushes with update reference callback', () {