Fix download all logs for pod with few containers (#7413)

* Use container name as downloaded file name

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Specify container to call for logs query

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Fixing tests

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

---------

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
This commit is contained in:
Alex Andreev authored and GitHub committed 2023-03-27 12:22:37 +03:00
1 parent 8dc0177cb2
commit 7d71873135
3 files changed
+5 -5

No files matched your search

@@ -190,12 +190,12 @@ describe("download logs options in logs dock tab", () => {
it("logs have been called with query", () => {
expect(callForLogsMock).toHaveBeenCalledWith(
{ name: "dockerExporter", namespace: "default" },
{ "previous": true, "timestamps": false },
{ "previous": true, "timestamps": false, container: "docker-exporter" },
);
});
it("shows save dialog with proper attributes", async () => {
expect(openSaveFileDialogMock).toHaveBeenCalledWith("dockerExporter.log", "all-logs", "text/plain");
expect(openSaveFileDialogMock).toHaveBeenCalledWith("docker-exporter.log", "all-logs", "text/plain");
});
it("doesn't block download dropdown for interaction after click", async () => {
@@ -265,7 +265,7 @@ describe("download logs options in logs dock tab", () => {
it("logs have been called", () => {
expect(callForLogsMock).toHaveBeenCalledWith(
{ name: "dockerExporter", namespace: "default" },
{ "previous": true, "timestamps": false },
{ "previous": true, "timestamps": false, container: "docker-exporter" },
);
});