Fix tests #285
Fix tests #285
Conversation
| </register> | ||
| <!-- Для использования экспорта нужно установить пакет NewPlatform.Flexberry.Reports.ExportToExcel --> | ||
| <!-- | ||
| <configSections> |
There was a problem hiding this comment.
Лучше вернуть отступы, чтобы потом не светилось, что в этом коммите эта строчка изменена.
И в целом в файле вроде не было изменений, лучше его вернуть к первоначальному виду.
| } | ||
| } | ||
| public static IUnityContainer _container; | ||
|
|
There was a problem hiding this comment.
public static IUnityContainer _container;
public static IServiceProvider _serviceProvider;
можно сделать private и не static вроде.
| @@ -14,13 +14,19 @@ namespace NewPlatform.Flexberry.ORM.ODataService.Tests | |||
| public class CustomWebApplicationFactory<TStartup> : WebApplicationFactory<TStartup> | |||
| where TStartup : class | |||
| { | |||
| public static Unity.IUnityContainer _unityContainer; | |||
There was a problem hiding this comment.
Нужен комментарий, почему здесь именно public static поле.
| public string CustomizationString => ""; | ||
|
|
||
| public virtual void ConfigureContainer(IUnityContainer unityContainer) |
There was a problem hiding this comment.
На основании https://habr.com/ru/sandbox/123391/ сформулировать, что это за метод, чтобы потом легче было разбираться.
| using Unity; | ||
|
|
||
| /// <summary> | ||
| /// Startup for tests. | ||
| /// </summary> | ||
| public class TestStartup : Startup | ||
| { | ||
| public static IUnityContainer _unityContainer; |
There was a problem hiding this comment.
Вроде можно private и не статичный.
| @@ -29,11 +32,17 @@ public TestStartup(IConfiguration configuration) | |||
| { | |||
| } | |||
|
|
|||
| public override void ConfigureContainer(IUnityContainer unityContainer) | |||
There was a problem hiding this comment.
На основании https://habr.com/ru/sandbox/123391/ сформулировать, что это за метод, чтобы потом легче было разбираться.
No description provided.