Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Fix some broken tests - seems they have been broken for a while! yet …

…the tests still passed?!?
  • Loading branch information...
commit 4bedc680ab024be2f432d2bd5347d71d00b69f99 1 parent 62fcfc8
@richardhopton richardhopton authored
Showing with 3 additions and 3 deletions.
  1. +3 −3 Simple.Data.InMemoryTest/InMemoryTests.cs
View
6 Simple.Data.InMemoryTest/InMemoryTests.cs
@@ -272,7 +272,7 @@ public void TestSkipAndTake()
public void TestJoin()
{
var adapter = new InMemoryAdapter();
- adapter.ConfigureJoin("Customer", "ID", "Orders", "Order", "CustomerID", "Customer");
+ adapter.ConfigureJoin("Customer", "ID", "Customer", "Order", "CustomerID", "Orders");
Database.UseMockAdapter(adapter);
var db = Database.Open();
db.Customer.Insert(ID: 1, Name: "NASA");
@@ -309,7 +309,7 @@ public void SeparateThreadsShouldSeeDifferentMocks()
public void FindAllWhenUsingNamePropertyShouldWork()
{
var adapter = new InMemoryAdapter();
- adapter.ConfigureJoin("Users", "Id", "Categories", "Categories", "UserId", "User");
+ adapter.ConfigureJoin("Users", "Id", "User", "Categories", "UserId", "Categories");
Database.UseMockAdapter(adapter);
var db = Database.Open();
@@ -327,7 +327,7 @@ public void FindAllWhenUsingNamePropertyShouldWork()
public void FindAllWhenUsingAnyOldPropertyNameShouldWork()
{
var adapter = new InMemoryAdapter();
- adapter.ConfigureJoin("Users", "Id", "Categories", "Categories", "UserId", "User");
+ adapter.ConfigureJoin("Users", "Id", "User", "Categories", "UserId", "Categories");
Database.UseMockAdapter(adapter);
var db = Database.Open();

5 comments on commit 4bedc68

@marcusoftnet

See?! Told you that the ConfigureJoin method was hard to grasp ;)

@markrendle
Owner

Yeah, yeah. Improved API on its way.

@richardhopton
Collaborator

Hmm - was looking at implementing joins in the query for in mem adapter but seems you maintain related tables during insert not on select.

@markrendle
Owner
@richardhopton
Collaborator

Not from what I could see... might me missing the point entirely tho!

Please sign in to comment.
Something went wrong with that request. Please try again.