Skip to content

06 my nunit web - #9

Open
ArtemNikit1n wants to merge 6 commits into
mainfrom
06-my-nunit-web
Open

06 my nunit web#9
ArtemNikit1n wants to merge 6 commits into
mainfrom
06-my-nunit-web

Conversation

@ArtemNikit1n

Copy link
Copy Markdown
Owner

No description provided.

@yurii-litvinov yurii-litvinov left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У меня при загрузке файла CORS говорит, что нельзя с localhost:3000 на localhost:5221 запросы слать, так что не работает :)

var tasks = assemblyPaths.Select(assemblyPath => ProcessAssemblyAsync(assemblyPath, testRun)).ToList();

await Task.WhenAll(tasks);
this.testRuns.TryAdd(testRun.Id, testRun);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Без базы данных это не очень интересно, требование условия "результаты всех тестовых прогонов, когда-либо запускавшихся на сервере" можно удовлетворить только очень условно (прогонов до перезапуска сервера).

formData.append('File', file);

try {
const response = await axios.post<UploadResponse>('http://localhost:5221/api/tests/upload', formData, {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хардкодить URL так нельзя, а то при деплое куда-то ещё клиент внезапно перестанет работать.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants