Skip to content

Commit 481f6f9

Browse files
Fix Code Factor errors
And slightly reduces/adjusts some changes
1 parent fa99e7f commit 481f6f9

File tree

15 files changed

+21
-51
lines changed

15 files changed

+21
-51
lines changed

src/NHibernate.Test/Any/AnyTypeTest.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,12 @@ protected override string CacheConcurrencyStrategy
2323
[Test]
2424
public void FlushProcessing()
2525
{
26-
Person person;
27-
Address address;
26+
var person = new Person();
27+
var address = new Address();
2828
//http://opensource.atlassian.com/projects/hibernate/browse/HHH-1663
2929
using (var session = OpenSession())
3030
using (var tran = session.BeginTransaction())
3131
{
32-
person = new Person();
33-
address = new Address();
3432
person.Data = address;
3533
session.SaveOrUpdate(person);
3634
session.SaveOrUpdate(address);

src/NHibernate.Test/Async/Any/AnyTypeTest.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,12 @@ protected override string CacheConcurrencyStrategy
3434
[Test]
3535
public async Task FlushProcessingAsync()
3636
{
37-
Person person;
38-
Address address;
37+
var person = new Person();
38+
var address = new Address();
3939
//http://opensource.atlassian.com/projects/hibernate/browse/HHH-1663
4040
using (var session = OpenSession())
4141
using (var tran = session.BeginTransaction())
4242
{
43-
person = new Person();
44-
address = new Address();
4543
person.Data = address;
4644
await (session.SaveOrUpdateAsync(person));
4745
await (session.SaveOrUpdateAsync(address));

src/NHibernate.Test/Async/ConnectionTest/AggressiveReleaseTest.cs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -240,15 +240,13 @@ public async Task ConnectionMaintanenceDuringFlushAsync()
240240
entities.Add(silly);
241241
await (s.SaveAsync(silly));
242242
}
243-
244243
await (s.FlushAsync());
245244

246245
foreach (Silly silly in entities)
247246
{
248247
silly.Name = "new-" + silly.Name;
249248
silly.Other.Name = "new-" + silly.Other.Name;
250249
}
251-
252250
// long initialCount = sessions.Statistics.getConnectCount();
253251
await (s.FlushAsync());
254252
//Assert.AreEqual(initialCount + 1, sessions.Statistics.getConnectCount(), "connection not maintained through Flush");
@@ -257,7 +255,6 @@ public async Task ConnectionMaintanenceDuringFlushAsync()
257255
await (s.DeleteAsync("from Other"));
258256
await (t.CommitAsync());
259257
}
260-
261258
Release(s);
262259
Done();
263260
}

src/NHibernate.Test/Async/DynamicEntity/Interceptor/InterceptorDynamicEntity.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,16 +37,14 @@ protected override void Configure(Configuration configuration)
3737
[Test]
3838
public async Task ItAsync()
3939
{
40-
Company company;
41-
Customer customer;
40+
var company = ProxyHelper.NewCompanyProxy();
41+
var customer = ProxyHelper.NewCustomerProxy();
4242
// Test saving these dyna-proxies
4343
using (var session = OpenSession())
4444
using (var tran = session.BeginTransaction())
4545
{
46-
company = ProxyHelper.NewCompanyProxy();
4746
company.Name = "acme";
4847
await (session.SaveAsync(company));
49-
customer = ProxyHelper.NewCustomerProxy();
5048
customer.Name = "Steve";
5149
customer.Company = company;
5250
await (session.SaveAsync(customer));

src/NHibernate.Test/Async/DynamicEntity/Tuplizer/TuplizerDynamicEntity.cs

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,31 +38,27 @@ protected override void Configure(Configuration configuration)
3838
[Test]
3939
public async Task ItAsync()
4040
{
41-
Company company;
42-
Customer customer;
43-
Address address;
44-
Person son, wife;
41+
var company = ProxyHelper.NewCompanyProxy();
42+
var customer = ProxyHelper.NewCustomerProxy();
43+
var address = ProxyHelper.NewAddressProxy();
44+
var son = ProxyHelper.NewPersonProxy();
45+
var wife = ProxyHelper.NewPersonProxy();
4546

4647
// Test saving these dyna-proxies
4748
using (var session = OpenSession())
4849
using (var tran = session.BeginTransaction())
4950
{
50-
company = ProxyHelper.NewCompanyProxy();
5151
company.Name = "acme";
5252
await (session.SaveAsync(company));
53-
customer = ProxyHelper.NewCustomerProxy();
5453
customer.Name = "Steve";
5554
customer.Company = company;
56-
address = ProxyHelper.NewAddressProxy();
5755
address.Street = "somewhere over the rainbow";
5856
address.City = "lawerence, kansas";
5957
address.PostalCode = "toto";
6058
customer.Address = address;
6159
customer.Family = new HashSet<Person>();
62-
son = ProxyHelper.NewPersonProxy();
6360
son.Name = "son";
6461
customer.Family.Add(son);
65-
wife = ProxyHelper.NewPersonProxy();
6662
wife.Name = "wife";
6763
customer.Family.Add(wife);
6864
await (session.SaveAsync(customer));

src/NHibernate.Test/Async/Generatedkeys/ByTrigger/GeneratedIdentityFixture.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ public async Task GetGeneratedKeysSupportAsync()
3737
using (var session = OpenSession())
3838
using (var tran = session.BeginTransaction())
3939
{
40-
4140
var e = new MyEntity { Name = "entity-1" };
4241
await (session.SaveAsync(e));
4342

src/NHibernate.Test/Async/ReadOnly/ReadOnlyProxyTest.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1721,7 +1721,6 @@ public async Task SetClosedSessionInLazyInitializerAsync()
17211721
Assert.That(s.Contains(dp), Is.True);
17221722
s.Close();
17231723

1724-
17251724
Assert.That(((INHibernateProxy) dp).HibernateLazyInitializer.Session, Is.Null);
17261725
Assert.That(((ISessionImplementor) s).IsClosed, Is.True);
17271726

src/NHibernate.Test/Async/ReadOnly/ReadOnlyVersionedNodes.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -640,7 +640,6 @@ protected override void OnTearDown()
640640
using (var s = OpenSession())
641641
using (var t = s.BeginTransaction())
642642
{
643-
644643
s.CreateQuery("delete from VersionedNode where parent is not null").ExecuteUpdate();
645644
s.CreateQuery("delete from VersionedNode").ExecuteUpdate();
646645

src/NHibernate.Test/ConnectionTest/AggressiveReleaseTest.cs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -228,15 +228,13 @@ public void ConnectionMaintanenceDuringFlush()
228228
entities.Add(silly);
229229
s.Save(silly);
230230
}
231-
232231
s.Flush();
233232

234233
foreach (Silly silly in entities)
235234
{
236235
silly.Name = "new-" + silly.Name;
237236
silly.Other.Name = "new-" + silly.Other.Name;
238237
}
239-
240238
// long initialCount = sessions.Statistics.getConnectCount();
241239
s.Flush();
242240
//Assert.AreEqual(initialCount + 1, sessions.Statistics.getConnectCount(), "connection not maintained through Flush");
@@ -245,7 +243,6 @@ public void ConnectionMaintanenceDuringFlush()
245243
s.Delete("from Other");
246244
t.Commit();
247245
}
248-
249246
Release(s);
250247
Done();
251248
}

src/NHibernate.Test/DynamicEntity/Interceptor/InterceptorDynamicEntity.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,16 +26,14 @@ protected override void Configure(Configuration configuration)
2626
[Test]
2727
public void It()
2828
{
29-
Company company;
30-
Customer customer;
29+
var company = ProxyHelper.NewCompanyProxy();
30+
var customer = ProxyHelper.NewCustomerProxy();
3131
// Test saving these dyna-proxies
3232
using (var session = OpenSession())
3333
using (var tran = session.BeginTransaction())
3434
{
35-
company = ProxyHelper.NewCompanyProxy();
3635
company.Name = "acme";
3736
session.Save(company);
38-
customer = ProxyHelper.NewCustomerProxy();
3937
customer.Name = "Steve";
4038
customer.Company = company;
4139
session.Save(customer);

src/NHibernate.Test/DynamicEntity/Tuplizer/TuplizerDynamicEntity.cs

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -27,31 +27,27 @@ protected override void Configure(Configuration configuration)
2727
[Test]
2828
public void It()
2929
{
30-
Company company;
31-
Customer customer;
32-
Address address;
33-
Person son, wife;
30+
var company = ProxyHelper.NewCompanyProxy();
31+
var customer = ProxyHelper.NewCustomerProxy();
32+
var address = ProxyHelper.NewAddressProxy();
33+
var son = ProxyHelper.NewPersonProxy();
34+
var wife = ProxyHelper.NewPersonProxy();
3435

3536
// Test saving these dyna-proxies
3637
using (var session = OpenSession())
3738
using (var tran = session.BeginTransaction())
3839
{
39-
company = ProxyHelper.NewCompanyProxy();
4040
company.Name = "acme";
4141
session.Save(company);
42-
customer = ProxyHelper.NewCustomerProxy();
4342
customer.Name = "Steve";
4443
customer.Company = company;
45-
address = ProxyHelper.NewAddressProxy();
4644
address.Street = "somewhere over the rainbow";
4745
address.City = "lawerence, kansas";
4846
address.PostalCode = "toto";
4947
customer.Address = address;
5048
customer.Family = new HashSet<Person>();
51-
son = ProxyHelper.NewPersonProxy();
5249
son.Name = "son";
5350
customer.Family.Add(son);
54-
wife = ProxyHelper.NewPersonProxy();
5551
wife.Name = "wife";
5652
customer.Family.Add(wife);
5753
session.Save(customer);

src/NHibernate.Test/Generatedkeys/ByTrigger/GeneratedIdentityFixture.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ public void GetGeneratedKeysSupport()
2626
using (var session = OpenSession())
2727
using (var tran = session.BeginTransaction())
2828
{
29-
3029
var e = new MyEntity { Name = "entity-1" };
3130
session.Save(e);
3231

src/NHibernate.Test/Generatedkeys/Identity/IdentityGeneratedKeysTest.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,18 +46,17 @@ public void IdentityColumnGeneratedIds()
4646
[Test, Ignore("Not supported yet.")]
4747
public void PersistOutsideTransaction()
4848
{
49-
MyEntity myEntity1, myEntity2;
49+
var myEntity1 = new MyEntity("test-save");
50+
var myEntity2 = new MyEntity("test-persist");
5051
using (var s = OpenSession())
5152
{
5253
// first test save() which should force an immediate insert...
53-
myEntity1 = new MyEntity("test-save");
5454
long id = (long) s.Save(myEntity1);
5555
Assert.IsNotNull(id, "identity column did not force immediate insert");
5656
Assert.AreEqual(id, myEntity1.Id);
5757

5858
// next test persist() which should cause a delayed insert...
5959
long initialInsertCount = Sfi.Statistics.EntityInsertCount;
60-
myEntity2 = new MyEntity("test-persist");
6160
s.Persist(myEntity2);
6261
Assert.AreEqual(
6362
initialInsertCount,

src/NHibernate.Test/ReadOnly/ReadOnlyProxyTest.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1819,7 +1819,6 @@ public void SetClosedSessionInLazyInitializer()
18191819
Assert.That(s.Contains(dp), Is.True);
18201820
s.Close();
18211821

1822-
18231822
Assert.That(((INHibernateProxy) dp).HibernateLazyInitializer.Session, Is.Null);
18241823
Assert.That(((ISessionImplementor) s).IsClosed, Is.True);
18251824

src/NHibernate.Test/ReadOnly/ReadOnlyVersionedNodes.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -212,11 +212,10 @@ public void UpdateSetReadOnlySetModifiableFailureExpected()
212212
[Ignore("Failure expected")]
213213
public void SetReadOnlyUpdateSetModifiableFailureExpected()
214214
{
215-
VersionedNode node;
215+
VersionedNode node = new VersionedNode("node", "node");
216216
using (var s = OpenSession())
217217
using (var t = s.BeginTransaction())
218218
{
219-
node = new VersionedNode("node", "node");
220219
s.Persist(node);
221220
t.Commit();
222221
s.Close();
@@ -707,7 +706,6 @@ protected override void OnTearDown()
707706
using (var s = OpenSession())
708707
using (var t = s.BeginTransaction())
709708
{
710-
711709
s.CreateQuery("delete from VersionedNode where parent is not null").ExecuteUpdate();
712710
s.CreateQuery("delete from VersionedNode").ExecuteUpdate();
713711

0 commit comments

Comments
 (0)