JasperFx/alba

A humble library of helpers and a Trie-based router for OWIN applications

Jeremy D. Miller

@jeremydmiller

@rytmis @jmezach And it’s up: https://jeremydmiller.com/2019/09/29/alba-3-1-supercharges-your-asp-net-core-http-contract-testing/

Thanks for all your work on this one.

Lauri Kotilainen

@rytmis

Yay! Thanks. 🙂

Jonathan Mezach

@jmezach

@jeremydmiller Thank you for letting us do this ;). Glad I could help out here.

Craig Quillen

@cquillen

Just getting started with Alba. Trying to test a Core MVC controller action.

It's failing with "bad request - invalid hostname"

Here is my SUT setup code:

       public async Task Test_Logging()

        {

            var builder = BusinessRulesEngine.Web.Program.CreateWebHostBuilder(new string[] { });



            using (var system = new SystemUnderTest(builder, typeof(Startup).Assembly))

            {

                await system.Scenario(_ =>

                {

                    _.Get.Url("/home/TestLogging");

                    _.ContentShouldBe("");

                    _.StatusCodeShouldBeOk();

                });



            }

        }

thoughts on why it doesn't like the hostname?

@jeremydmiller

Jeremy D. Miller

@jeremydmiller

I have no idea, I’ve never seen that before. Can you post the actual stacktrace? Something tripping up inside of AspNetCore itself I’d expect. Something in your code looking for the host name off of the HttpContext?

Craig Quillen

@cquillen

I actually just figured it out.

had mixed runtime refs between app and test projects.

added these to test.csproj

Jeremy D. Miller

@jeremydmiller

AspNetCore 2.*? I think that’s in the Alba documentation isn’t it?

Not an issue w/ AspNetCore 3.0 I believe

Craig Quillen

@cquillen

and <project sdk="microsoft.net.sdk.web"

yeah 2.2

i was also getting the multiple version warning during assembly binding

all better now

not sure about the docs. I'll look again.

now i'm getting authorization failures, but that I understand. I've not done anything to set an auth header for my test request.

pretty sure it's working

Jeremy D. Miller

@jeremydmiller

Cool, and feel free to yell back here about any holes in the docs

For 3.0, they changed things for Nuget references for AspNetCore yet again. Alba just targets netcoreapp3.0 and the references will float based on the runtime version now. Should soften up the exact problems you saw

Tomasz Jaskula

@tjaskula

Hey, I started to use Alba and it's really cool, however there is one thing I would like to ask for. BeforeEach actions passes in not fully initialized HttpContext. For example RequestedServices collection is uninitialized which would be useful in some cases. WDYT ?

Jeremy D. Miller

@jeremydmiller

That’s filled in by ASP.net Core itself as part of the request. That’s actually done by middleware in the execution pipeline

Tomasz Jaskula

@tjaskula

Ok, that means that there is no mean to resolve a service before the request ?

Jeremy D. Miller

@jeremydmiller

Not from the request’s own scope container. You can from the root application container

Tomasz Jaskula

@tjaskula

I have another issue as well, in some cases the HttpContext handed over to IScenarioAssertion has RequestServices == null. Is this something you've noticed ?

Jeremy D. Miller

@jeremydmiller

I can’t say that I have, no, but again, Alba itself depends on ASP.Net Core to deal with that. What are you needing to do? Why does it need to be the request scoped services?

Tomasz Jaskula

@tjaskula

I have this code to assert on GraphQL queries

public class SuccessResultAssertion : GraphQLAssertion
    {
        private readonly string _result;

        public SuccessResultAssertion(string result) => _result = result;

        public override void Assert(Scenario scenario, HttpContext context, ScenarioAssertionException ex)
        {
            var writer = (IDocumentWriter) context.RequestServices.GetService(typeof(IDocumentWriter));
            var expectedResult = writer.Write(CreateQueryResult(_result));

            var body = ex.ReadBody(context);
            if (!body.Equals(expectedResult))
            {
                ex.Add($"Expected '{expectedResult}' but got '{body}'");
            }
        }
    }

So I noticed that while executing tests only the first test has context.RequestServices filled, the scond tests throws an exception as RequestServices is null

GraphQLAssertion is abstract class implementing IScenarioAssertion

Jeremy D. Miller

@jeremydmiller

How is IDocumentWriter scoped? Can you not pull it out of the application services instead?

Tomasz Jaskula

@tjaskula

how can I access the application service container from here ?

Jeremy D. Miller

@jeremydmiller

Trying to figure that out myself;)

Sorry, gotta go finish getting kids ready for school. I’ll be back later today

Tomasz Jaskula

@tjaskula

ok, thanks :)

Jeremy D. Miller

@jeremydmiller

Wait, duh, it’s IHost.Services that you can get to off the SystemUnderTest

Tomasz Jaskula

@tjaskula

Ok so I'll have to pass it somehow to the assertion

Tomasz Jaskula

@tjaskula

After looking a bit, there is no mean I can access IHost.Services from inside the IScenarioAssertion. Scenario passed in to Assert method would need to expose the SystemUnderTests but it's not the case. It's kept internally

Jeremy D. Miller

@jeremydmiller

We could do a quick Alba 4.1 and add an overload of some kind to make it easier to do this kind of thing

And what Graph QL toolkit are you using out of curiosity? Is it the one from @joemcbride ?

Tomasz Jaskula

@tjaskula

That would be awesome if we could access somehow the app services container as we cannot rely on HttpContext. I'm using https://github.com/graphql-dotnet/graphql-dotnet

Jeremy D. Miller

@jeremydmiller

Cool, Joe is a former colleague of mine. He helped with Alba in the early days too

Tomasz Jaskula

@tjaskula

Oh, that's awesome. This world is so small

Do you think there is a chance for a release of Alba with this feature anytime soon then ?

Tomasz Jaskula

@tjaskula

@jeremydmiller I've opened the issue for reference JasperFx/alba#69

Jeremy D. Miller

@jeremydmiller

With me having 2 weeks off for Christmas, yes;)

Where communities thrive

People

Repo info

Activity