Base solution for your next web application
Open Closed

8.8.0 upgrade - Should_Change_Email_Settings test fails #9133


User avatar
0
sneddo created

Hi, I have upgraded to 8.8 and everything appears to have gone well apart from the above test now failing.

This is the message:

Message: Abp.AbpException : There is no setting defined with name: ExternalLoginProvider.Facebook Stack Trace: SettingDefinitionManager.GetSettingDefinition(String name) SettingManager.GetSettingValueInternalAsync(String name, Nullable1 tenantId, Nullable1 userId, Boolean fallbackToDefault) UnitOfWorkInterceptor.InternalInterceptAsynchronous[TResult](IInvocation invocation) HostSettingsAppService.GetExternalLoginProviderSettings() line 212 HostSettingsAppService.GetAllSettings() line 48 AuthorizationInterceptor.InternalInterceptAsynchronous[TResult](IInvocation invocation) UnitOfWorkInterceptor.InternalInterceptAsynchronous[TResult](IInvocation invocation) AuditingInterceptor.InternalInterceptAsynchronous[TResult](IInvocation invocation) AuditingInterceptor.InternalInterceptAsynchronous[TResult](IInvocation invocation) ValidationInterceptor.InternalInterceptAsynchronous[TResult](IInvocation invocation) HostSettingsAppService_EmailSettings_Test.Should_Change_Email_Settings() line 44 --- End of stack trace from previous location where exception was thrown ---

This makes me think I might have missed something during the upgrade but I cant for the life of me find what that might be. It appears the setting provider _hostSettingsAppService.GetAllSettings() isnt being prepared with the correct data before the test is run.


1 Answer(s)
  • User Avatar
    0
    musa.demir created

    Hi @sneddo Changing that lines should solve your problem. https://github.com/aspnetzero/aspnet-zero-core/commit/521e2d4c5cc7c11e61de538573b600fec804ae08#diff-d1f06df1abdd1d79effb0f032311493cR970