Skip to content

refactor(http): Improves base64 encoding/decoding with feature detection #67002

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
SkyZeroZx wants to merge 5 commits into
base: main
Choose a base branch
from
Open

refactor(http): Improves base64 encoding/decoding with feature detection #67002

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
2acd6ad
fix(http): correctly cache blob responses in transfer cache
SkyZeroZx Feb 11, 2026
6a0a9ed
fixup! fix(http): correctly cache blob responses in transfer cache
SkyZeroZx Feb 12, 2026
e9eaf18
fixup! fix(http): correctly cache blob responses in transfer cache
SkyZeroZx Feb 13, 2026
63f083d
fixup! fix(http): correctly cache blob responses in transfer cache
SkyZeroZx Feb 18, 2026
8312083
test(http): correct response handling in httpResource
SkyZeroZx Mar 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
test(http): correct response handling in httpResource 
Ensure httpResource properly handles the response flow
  • Loading branch information
@SkyZeroZx
SkyZeroZx committed Mar 4, 2026
commit 83120839189edcb3b747697f48b58cccc61ddf4b
12 changes: 4 additions & 8 deletions packages/common/http/test/resource_spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ import {
import {HttpTestingController, provideHttpClientTesting} from '../testing';
import {withHttpTransferCache} from '../src/transfer_cache';
import {HttpClient} from '../src/client';
import {lastValueFrom} from 'rxjs';

describe('httpResource', () => {
beforeEach(() => {
Expand Down Expand Up @@ -424,15 +425,10 @@ describe('httpResource', () => {

it('should synchronously resolve with a cached value from TransferState', async () => {
Copy link
Copy Markdown
Contributor Author

@SkyZeroZx SkyZeroZx Mar 4, 2026

Choose a reason for hiding this comment

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

According to my review, this test was failing due to the recent changes to the transfer cache, which now behaves asynchronously.
I'm not sure if we should change the test name or if it's correct.

Copy link
Copy Markdown
Contributor

@thePunderWoman thePunderWoman Mar 4, 2026

Choose a reason for hiding this comment

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

This change was intentional to ensure there's no loading state when pulling data from the transfer cache. #67307

Copy link
Copy Markdown
Contributor Author

@SkyZeroZx SkyZeroZx Mar 4, 2026

Choose a reason for hiding this comment

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

So I think it would stay as it is, only keeping the change I made due to the TransferCache update

Copy link
Copy Markdown
Contributor Author

@SkyZeroZx SkyZeroZx Mar 13, 2026

Choose a reason for hiding this comment

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

@thePunderWoman With the latest changes, I think it would be okay to merge, or would we still need to adjust something?

globalThis['ngServerMode'] = true;
let requestResolved = false;
TestBed.inject(HttpClient)
.get('/data')
.subscribe(() => (requestResolved = true));
const req = TestBed.inject(HttpTestingController).expectOne('/data');
req.flush([1, 2, 3]);

expect(requestResolved).toBe(true);

const response = lastValueFrom(TestBed.inject(HttpClient).get('/data'));
TestBed.inject(HttpTestingController).expectOne('/data').flush([1, 2, 3]);
await response;
// Now switch to client mode
globalThis['ngServerMode'] = false;

Expand Down