C#•3y ago

❔ want to add a conditional .Take(10) to a query without duplicating the code

Im looking for an easy way to add a .Take(10) or .Take(x) to a iqueryable so that my test go faster instead of running on the entire data.

is there a way to make it so if x is null it takes all or ignres the take() option ??

Head0nF1re•6/21/23, 6:27 PM

"so that my test go faster instead of running on the entire data". What do you mean?

PatrickGOP•6/21/23, 6:29 PM

well lets say my code is
context.Customers.Tolist().Foreach( DO A LOT OF STUFF )
I would like to add a .Take(10) but only if i passed an argument that says to run this in test

i could do
if (isTest){
context.Customers.Take(10).Tolist().Foreach( DO A LOT OF STUFF )
}
else{
context.Customers.Tolist().Foreach( DO A LOT OF STUFF )
}

but i have to change tons of queries of this kind so I was wondering if theres a way to put the Take() and basically takeAll unless otherwise specified

Pobiega•6/21/23, 6:31 PM

PatrickGOP•6/21/23, 6:32 PM

ok so its not builtin

Pobiega•6/21/23, 6:32 PM

no, ofc not

Pobiega•6/21/23, 6:32 PM

its a bit weird

Pobiega•6/21/23, 6:32 PM

Head0nF1re•6/21/23, 6:32 PM

Do ToList() after the ifs, just to see what happens

Head0nF1re•6/21/23, 6:32 PM

Also, weird requirement that "test" thing

HHead0nF1re Do ToList() after the ifs, just to see what happens

PatrickGOP•6/21/23, 6:33 PM

the thing is i dont want a if lol

Pobiega•6/21/23, 6:34 PM

so use the extension method provided above

Pobiega•6/21/23, 6:34 PM

but its a bit weird. in general your approach should be that tests provide their own data

PatrickGOP•6/21/23, 6:34 PM

yea that's basically what I wanted but I thought there might have been something of this kind already existing in LINQ

PPatrickG the thing is i dont want a if lol

Head0nF1re•6/21/23, 6:35 PM

It's just for you to understand what happens. Since in your example you did multiple ToList(), I don't think you understand what fundamentally happens

PatrickGOP•6/21/23, 6:36 PM

its ok dont worry about it

PPobiega so use the extension method provided above

PatrickGOP•6/21/23, 6:37 PM

its not a format unit test, im just gonna change it to 'test mode' and run it to see something

Pobiega•6/21/23, 6:37 PM

i see

PatrickGOP•6/21/23, 6:37 PM

formal*

Accord•6/22/23, 6:38 PM

Was this issue resolved? If so, run

/close

/close

otherwise I will mark this as stale and this post will be archived until there is new activity.