Skip to content
This repository was archived by the owner on Dec 24, 2022. It is now read-only.

Commit 80e559f

Browse files
committed
Fix Can_OrderBy_Fields_with_different_sort_directions test on Oracle.
1 parent 4351599 commit 80e559f

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

tests/ServiceStack.OrmLite.Tests/Expression/SqlExpressionTests.cs

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
using System;
22
using System.Collections.Generic;
33
using System.Data;
4+
using System.Linq;
45
using NUnit.Framework;
56
using ServiceStack.DataAnnotations;
67
using ServiceStack.Text;
@@ -250,22 +251,22 @@ public void Can_OrderBy_Fields_with_different_sort_directions()
250251
db.DropAndCreateTable<LetterFrequency>();
251252
db.DropAndCreateTable<LetterStat>();
252253

253-
var i = 0;
254+
var insertedIds = new List<long>();
254255
"A,B,B,C,C,C,D,D,E".Split(',').Each(letter => {
255-
db.Insert(new LetterFrequency { Letter = letter }, selectIdentity: true);
256+
insertedIds.Add(db.Insert(new LetterFrequency { Letter = letter }, selectIdentity: true));
256257
});
257258

258259
var rows = db.Select<LetterFrequency>(q => q.OrderByFields("Letter", "Id"));
259260
Assert.That(rows.Map(x => x.Letter), Is.EquivalentTo("A,B,B,C,C,C,D,D,E".Split(',')));
260-
Assert.That(rows.Map(x => x.Id), Is.EquivalentTo("1,2,3,4,5,6,7,8,9".Split(',').Map(int.Parse)));
261+
Assert.That(rows.Map(x => x.Id), Is.EquivalentTo(insertedIds));
261262

262263
rows = db.Select<LetterFrequency>(q => q.OrderByFields("Letter", "-Id"));
263264
Assert.That(rows.Map(x => x.Letter), Is.EquivalentTo("A,B,B,C,C,C,D,D,E".Split(',')));
264-
Assert.That(rows.Map(x => x.Id), Is.EquivalentTo("1,3,2,6,5,4,8,7,9".Split(',').Map(int.Parse)));
265+
Assert.That(rows.Map(x => x.Id), Is.EquivalentTo(insertedIds));
265266

266267
rows = db.Select<LetterFrequency>(q => q.OrderByFieldsDescending("Letter", "-Id"));
267268
Assert.That(rows.Map(x => x.Letter), Is.EquivalentTo("E,D,D,C,C,C,B,B,A".Split(',')));
268-
Assert.That(rows.Map(x => x.Id), Is.EquivalentTo("9,7,8,4,5,6,2,3,1".Split(',').Map(int.Parse)));
269+
Assert.That(rows.Map(x => x.Id), Is.EquivalentTo(Enumerable.Reverse(insertedIds)));
269270
}
270271

271272
}

0 commit comments

Comments
 (0)