nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: mlaursen/react-md
Pull request #752 (2018-06-01 00:27:31)

SelectField - Use parseFloat for numbers to allow decimals to work
Fixes for https://github.com/mlaursen/react-md/issues/751


Patch:
diff --git a/src/js/SelectFields/SelectField.js b/src/js/SelectFields/SelectField.js
index 38a2b79389..197071fb51 100644
--- a/src/js/SelectFields/SelectField.js
+++ b/src/js/SelectFields/SelectField.js
@@ -650,7 +650,7 @@ export default class SelectField extends PureComponent {
     const v = this._getItemPart(item, itemLabel, itemValue);
     const label = this._getItemPart(item, itemLabel, itemValue, true);
 
-    return v === value || v === parseInt(value, 10) ? label : '';
+    return v === value || v === parseFloat(value) ? label : '';
   };
 
   _getActive = (props, state) => {
@@ -970,7 +970,7 @@ export default class SelectField extends PureComponent {
     const dataValue = this._getItemPart(item, itemLabel, itemValue);
     const primaryText = this._getItemPart(item, itemLabel, itemValue, true);
 
-    const active = dataValue === value || dataValue === parseInt(value, 10);
+    const active = dataValue === value || dataValue === parseFloat(value);
     const stripped = (typeof stripActiveItem !== 'undefined' ? stripActiveItem : below) && active;
     if (!stripped) {
       const objectType = typeof item === 'object';


Test patch:
diff --git a/src/js/SelectFields/__tests__/SelectField.js b/src/js/SelectFields/__tests__/SelectField.js
index b85e265574..1976cd6797 100644
--- a/src/js/SelectFields/__tests__/SelectField.js
+++ b/src/js/SelectFields/__tests__/SelectField.js
@@ -79,6 +79,9 @@ describe('SelectField', () => {
 
       expect(field._getItemPart({ label: 0, value: 'a' }, 'label', 'value', false)).toBe('a');
       expect(field._getItemPart({ label: 0, value: 'a' }, 'label', 'value', true)).toBe(0);
+
+      expect(field._getItemPart({ label: 1.5, value: 'a' }, 'label', 'value', false)).toBe('a');
+      expect(field._getItemPart({ label: 1.5, value: 'a' }, 'label', 'value', true)).toBe(1.5);
     });
   });
 
@@ -103,6 +106,8 @@ describe('SelectField', () => {
       expect(field._getActiveItemLabel({ label: 'No Way', value: 0 }, '', 'label', 'value')).toBe('');
       expect(field._getActiveItemLabel({ label: 'No Way', value: '0' }, '', 'label', 'value')).toBe('');
       expect(field._getActiveItemLabel({ label: 'No Way', value: '' }, '', 'label', 'value')).toBe('No Way');
+      expect(field._getActiveItemLabel({ label: 'No Way', value: 3.5 }, '', 'label', 'value')).toBe('');
+      expect(field._getActiveItemLabel({ label: 'No Way', value: 3.5 }, '3.5', 'label', 'value')).toBe('No Way');
     });
   });
Repository: gitify-app/gitify
Pull request #822 (2024-02-25 22:24:00)

feat: issue not planned icon


Patch:
diff --git a/src/utils/github-api.ts b/src/utils/github-api.ts
index 2fa8d09f..bd79b5b4 100644
--- a/src/utils/github-api.ts
+++ b/src/utils/github-api.ts
@@ -114,6 +114,8 @@ export function getNotificationTypeIcon(
         case 'closed':
         case 'completed':
           return IssueClosedIcon;
+        case 'not_planned':
+          return SkipIcon;
         case 'reopened':
           return IssueReopenedIcon;
         default:


Test patch:
diff --git a/src/utils/github-api.test.ts b/src/utils/github-api.test.ts
index 019f3c4e..d3a335b2 100644
--- a/src/utils/github-api.test.ts
+++ b/src/utils/github-api.test.ts
@@ -96,6 +96,14 @@ describe('getNotificationTypeIcon', () => {
         }),
       ).displayName,
     ).toBe('IssueClosedIcon');
+    expect(
+      getNotificationTypeIcon(
+        createSubjectMock({
+          type: 'Issue',
+          state: 'not_planned',
+        }),
+      ).displayName,
+    ).toBe('SkipIcon');
     expect(
       getNotificationTypeIcon(
         createSubjectMock({
Repository: gitify-app/gitify
Pull request #1060 (2024-04-23 14:50:25)

test: rename suite
Looks like I missed this when renaming `settings.colors` to `settings.detailedNotifications` 🙈 


Patch:
diff --git a/src/utils/api/client.ts b/src/utils/api/client.ts
index c0b02b7c..0849f7e7 100644
--- a/src/utils/api/client.ts
+++ b/src/utils/api/client.ts
@@ -114,7 +114,7 @@ export function ignoreNotificationThreadSubscription(
 ): AxiosPromise<NotificationThreadSubscription> {
   const baseUrl = getGitHubAPIBaseUrl(hostname);
   const url = new URL(
-    `${baseUrl}/notifications/threads/${threadId}/subscriptions`,
+    `${baseUrl}/notifications/threads/${threadId}/subscription`,
   );
   return apiRequestAuth(url.toString(), 'PUT', token, { ignored: true });
 }


Test patch:
diff --git a/src/hooks/useNotifications.test.ts b/src/hooks/useNotifications.test.ts
index 6eb889fc..f8c7bb16 100644
--- a/src/hooks/useNotifications.test.ts
+++ b/src/hooks/useNotifications.test.ts
@@ -216,8 +216,8 @@ describe('hooks/useNotifications.ts', () => {
       });
     });
 
-    describe('with colors', () => {
-      it('should fetch notifications with success - with colors', async () => {
+    describe('with detailed notifications', () => {
+      it('should fetch notifications with success', async () => {
         const accounts: AuthState = {
           ...mockAccounts,
           enterpriseAccounts: [],
diff --git a/src/utils/api/client.test.ts b/src/utils/api/client.test.ts
index f0cda8ba..6599dc38 100644
--- a/src/utils/api/client.test.ts
+++ b/src/utils/api/client.test.ts
@@ -219,7 +219,7 @@ describe('utils/api/client.ts', () => {
       );
 
       expect(axios).toHaveBeenCalledWith({
-        url: `https://api.github.com/notifications/threads/${mockThreadId}/subscriptions`,
+        url: `https://api.github.com/notifications/threads/${mockThreadId}/subscription`,
         method: 'PUT',
         data: { ignored: true },
       });
@@ -235,7 +235,7 @@ describe('utils/api/client.ts', () => {
       );
 
       expect(axios).toHaveBeenCalledWith({
-        url: `https://example.com/api/v3/notifications/threads/${mockThreadId}/subscriptions`,
+        url: `https://example.com/api/v3/notifications/threads/${mockThreadId}/subscription`,
         method: 'PUT',
         data: { ignored: true },
       });
Repository: gitify-app/gitify
Pull request #1075 (2024-05-01 19:44:16)

feat: skip fetching PR comments when none exist
Pull Request notifications often have a `subject.latest_comment_url` the same as their `subject.url`, which is when no PR comments have been made.

This enhancement avoids an unnecessary API call under these conditions.


Patch:
diff --git a/src/utils/subject.ts b/src/utils/subject.ts
index dbf45ada..a172ad6b 100644
--- a/src/utils/subject.ts
+++ b/src/utils/subject.ts
@@ -224,7 +224,10 @@ async function getGitifySubjectForPullRequest(
 
   let prCommentUser: User;
 
-  if (notification.subject.latest_comment_url) {
+  if (
+    notification.subject.latest_comment_url &&
+    notification.subject.latest_comment_url !== notification.subject.url
+  ) {
     const prComment = (
       await getIssueOrPullRequestComment(
         notification.subject.latest_comment_url,


Test patch:
diff --git a/src/utils/subject.test.ts b/src/utils/subject.test.ts
index bb2aaee2..cb279860 100644
--- a/src/utils/subject.test.ts
+++ b/src/utils/subject.test.ts
@@ -736,6 +736,35 @@ describe('utils/subject.ts', () => {
         });
       });
 
+      it('avoid fetching comments if latest_comment_url and url are the same', async () => {
+        mockNotification.subject.latest_comment_url =
+          mockNotification.subject.url;
+
+        nock('https://api.github.com')
+          .get('/repos/gitify-app/notifications-test/pulls/1')
+          .reply(200, {
+            state: 'open',
+            draft: false,
+            merged: false,
+            user: mockAuthor,
+          });
+
+        const result = await getGitifySubjectDetails(
+          mockNotification,
+          mockAccounts.token,
+        );
+
+        expect(result).toEqual({
+          state: 'open',
+          user: {
+            login: mockAuthor.login,
+            html_url: mockAuthor.html_url,
+            avatar_url: mockAuthor.avatar_url,
+            type: mockAuthor.type,
+          },
+        });
+      });
+
       it('handle pull request without latest_comment_url', async () => {
         mockNotification.subject.latest_comment_url = null;
Repository: auth0/auth0-python
Pull request #301 (2022-01-18 13:35:46)

[SDK-3174] Add `DELETE` method for `/api/v2/users/{id}/authenticators` endpoint
Otherwise, we have to get all enrolments and delete them one by one

### Changes

Added a missing method on users from the management api [delete_authenticators](https://auth0.com/docs/api/management/v2#!/Users/delete_authenticators)

### References

None, I need this personally.

### Testing

- [x] This change adds unit test coverage
- [ ] This change adds integration test coverage
- [x] This change has been tested on the latest version of the platform/language or why not

### Checklist

- [x] I have read the [Auth0 general contribution guidelines](https://github.com/auth0/open-source-template/blob/master/GENERAL-CONTRIBUTING.md)
- [x] I have read the [Auth0 Code of Conduct](https://github.com/auth0/open-source-template/blob/master/CODE-OF-CONDUCT.md)
- [x] All existing and new tests complete without errors



Patch:
diff --git a/auth0/v3/management/users.py b/auth0/v3/management/users.py
index 3686d08..fb8f2c1 100644
--- a/auth0/v3/management/users.py
+++ b/auth0/v3/management/users.py
@@ -298,6 +298,17 @@ class Users(object):
         url = self._url('{}/multifactor/{}'.format(id, provider))
         return self.client.delete(url)
 
+    def delete_authenticators(self, id):
+        """Delete a user's MFA enrollments.
+
+        Args:
+            id (str): The user's id.
+
+        See: https://auth0.com/docs/api/management/v2#!/Users/delete_authenticators
+        """
+        url = self._url('{}/authenticators'.format(id))
+        return self.client.delete(url)
+
     def unlink_user_account(self, id, provider, user_id):
         """Unlink a user account
 


Test patch:
diff --git a/auth0/v3/test/management/test_users.py b/auth0/v3/test/management/test_users.py
index cbefbbe..5d566ab 100644
--- a/auth0/v3/test/management/test_users.py
+++ b/auth0/v3/test/management/test_users.py
@@ -260,6 +260,17 @@ class TestUsers(unittest.TestCase):
             'https://domain/api/v2/users/an-id/multifactor/provider'
         )
 
+    @mock.patch('auth0.v3.management.users.RestClient')
+    def test_delete_authenticators(self, mock_rc):
+        mock_instance = mock_rc.return_value
+
+        u = Users(domain='domain', token='jwttoken')
+        u.delete_authenticators('an-id')
+
+        mock_instance.delete.assert_called_with(
+            'https://domain/api/v2/users/an-id/authenticators'
+        )
+
     @mock.patch('auth0.v3.management.users.RestClient')
     def test_unlink_user_account(self, mock_rc):
         mock_instance = mock_rc.return_value
Repository: gitify-app/gitify
Pull request #1186 (2024-06-05 16:53:57)

fix: tooltip z index to prevent items showing through
**Before**
![Screenshot 2024-06-05 at 12 53 38 PM](https://github.com/gitify-app/gitify/assets/386277/7cb6d8bb-dd47-41ed-a8f1-103668a8cbb9)


**After**
![Screenshot 2024-06-05 at 12 53 09 PM](https://github.com/gitify-app/gitify/assets/386277/8e821506-dc22-4339-a0dc-5ae4b2c7640c)



Patch:
diff --git a/src/components/fields/Tooltip.tsx b/src/components/fields/Tooltip.tsx
index fdf89223..fa4e5517 100644
--- a/src/components/fields/Tooltip.tsx
+++ b/src/components/fields/Tooltip.tsx
@@ -19,7 +19,7 @@ export const Tooltip: FC<ITooltip> = (props: ITooltip) => {
     >
       <QuestionIcon className="text-blue-500 ml-1" />
       {showTooltip && (
-        <div className="absolute bg-white dark:bg-gray-sidebar border border-gray-300 rounded p-2 shadow">
+        <div className="absolute bg-white dark:bg-gray-sidebar border border-gray-300 rounded p-2 shadow z-10">
           <div className="text-gray-700 dark:text-white text-xs text-left">
             {props.tooltip}
           </div>


Test patch:
diff --git a/src/components/fields/__snapshots__/Tooltip.test.tsx.snap b/src/components/fields/__snapshots__/Tooltip.test.tsx.snap
index 2fe780fc..d8c4bad3 100644
--- a/src/components/fields/__snapshots__/Tooltip.test.tsx.snap
+++ b/src/components/fields/__snapshots__/Tooltip.test.tsx.snap
@@ -21,7 +21,7 @@ exports[`components/fields/Tooltip.tsx should display on mouse enter / leave 1`]
     />
   </svg>
   <div
-    class="absolute bg-white dark:bg-gray-sidebar border border-gray-300 rounded p-2 shadow"
+    class="absolute bg-white dark:bg-gray-sidebar border border-gray-300 rounded p-2 shadow z-10"
   >
     <div
       class="text-gray-700 dark:text-white text-xs text-left"
Repository: gitify-app/gitify
Pull request #998 (2024-04-11 13:43:03)

fix: logout from


Patch:
diff --git a/.husky/pre-commit b/.husky/pre-commit
index c6254913..db4bafce 100755
--- a/.husky/pre-commit
+++ b/.husky/pre-commit
@@ -1,2 +1,2 @@
 pnpx lint-staged
-pnpm test -- --onlyChanged
+pnpm test
\ No newline at end of file
diff --git a/src/routes/Settings.tsx b/src/routes/Settings.tsx
index 89eb56c2..ddc94a44 100644
--- a/src/routes/Settings.tsx
+++ b/src/routes/Settings.tsx
@@ -244,7 +244,7 @@ export const SettingsRoute: FC = () => {
           <button
             type="button"
             className={footerButtonClass}
-            title={`Logout ${accounts.user.login}`}
+            title={`Logout from ${accounts.user.login}`}
             role="button"
             onClick={logoutUser}
           >


Test patch:
diff --git a/src/routes/Settings.test.tsx b/src/routes/Settings.test.tsx
index 26ba42a9..eefc923b 100644
--- a/src/routes/Settings.test.tsx
+++ b/src/routes/Settings.test.tsx
@@ -64,7 +64,7 @@ describe('routes/Settings.tsx', () => {
       );
     });
 
-    fireEvent.click(screen.getByTitle('Logout octocat'));
+    fireEvent.click(screen.getByTitle('Logout from octocat'));
 
     expect(logoutMock).toHaveBeenCalledTimes(1);
 
diff --git a/src/routes/__snapshots__/Settings.test.tsx.snap b/src/routes/__snapshots__/Settings.test.tsx.snap
index 3948a30f..d871c688 100644
--- a/src/routes/__snapshots__/Settings.test.tsx.snap
+++ b/src/routes/__snapshots__/Settings.test.tsx.snap
@@ -416,7 +416,7 @@ exports[`routes/Settings.tsx should render itself & its children 1`] = `
       <button
         class="hover:text-gray-500 py-1 px-2 my-1 mx-2 focus:outline-none"
         role="button"
-        title="Logout octocat"
+        title="Logout from octocat"
         type="button"
       >
         <svg
Repository: gitify-app/gitify
Pull request #1413 (2024-07-25 20:29:30)

refactor: rename isEnterpriseHost helper
Clarify helper is for checking if a hostname is for GitHub Enterprise Server, rather than GitHub Cloud (free, enterprise, etc)


Patch:
diff --git a/src/utils/api/utils.ts b/src/utils/api/utils.ts
index 52fdf25f..644d9f33 100644
--- a/src/utils/api/utils.ts
+++ b/src/utils/api/utils.ts
@@ -1,11 +1,11 @@
 import type { Hostname } from '../../types';
 import Constants from '../constants';
-import { isEnterpriseHost } from '../helpers';
+import { isEnterpriseServerHost } from '../helpers';
 
 export function getGitHubAPIBaseUrl(hostname: Hostname): URL {
   const url = new URL(Constants.GITHUB_API_BASE_URL);
 
-  if (isEnterpriseHost(hostname)) {
+  if (isEnterpriseServerHost(hostname)) {
     url.hostname = hostname;
     url.pathname = '/api/v3/';
   }
@@ -15,7 +15,7 @@ export function getGitHubAPIBaseUrl(hostname: Hostname): URL {
 export function getGitHubGraphQLUrl(hostname: Hostname): URL {
   const url = new URL(Constants.GITHUB_API_GRAPHQL_URL);
 
-  if (isEnterpriseHost(hostname)) {
+  if (isEnterpriseServerHost(hostname)) {
     url.hostname = hostname;
     url.pathname = '/api/graphql';
   }
diff --git a/src/utils/helpers.ts b/src/utils/helpers.ts
index 31f047f0..a0724bfe 100644
--- a/src/utils/helpers.ts
+++ b/src/utils/helpers.ts
@@ -17,7 +17,7 @@ export function getPlatformFromHostname(hostname: string): PlatformType {
     : 'GitHub Enterprise Server';
 }
 
-export function isEnterpriseHost(hostname: Hostname): boolean {
+export function isEnterpriseServerHost(hostname: Hostname): boolean {
   return !hostname.endsWith(Constants.DEFAULT_AUTH_OPTIONS.hostname);
 }
 


Test patch:
diff --git a/src/utils/helpers.test.ts b/src/utils/helpers.test.ts
index 49c83bfe..805c740b 100644
--- a/src/utils/helpers.test.ts
+++ b/src/utils/helpers.test.ts
@@ -16,7 +16,7 @@ import {
   generateNotificationReferrerId,
   getFilterCount,
   getPlatformFromHostname,
-  isEnterpriseHost,
+  isEnterpriseServerHost,
 } from './helpers';
 
 describe('utils/helpers.ts', () => {
@@ -40,15 +40,19 @@ describe('utils/helpers.ts', () => {
     });
   });
 
-  describe('isEnterpriseHost', () => {
+  describe('isEnterpriseServerHost', () => {
     it('should return true for enterprise host', () => {
-      expect(isEnterpriseHost('github.gitify.app' as Hostname)).toBe(true);
-      expect(isEnterpriseHost('api.github.gitify.app' as Hostname)).toBe(true);
+      expect(isEnterpriseServerHost('github.gitify.app' as Hostname)).toBe(
+        true,
+      );
+      expect(isEnterpriseServerHost('api.github.gitify.app' as Hostname)).toBe(
+        true,
+      );
     });
 
     it('should return false for non-enterprise host', () => {
-      expect(isEnterpriseHost('github.com' as Hostname)).toBe(false);
-      expect(isEnterpriseHost('api.github.com' as Hostname)).toBe(false);
+      expect(isEnterpriseServerHost('github.com' as Hostname)).toBe(false);
+      expect(isEnterpriseServerHost('api.github.com' as Hostname)).toBe(false);
     });
   });
Repository: gitify-app/gitify
Pull request #1309 (2024-06-22 15:07:59)

refactor: use URL to build href


Patch:
diff --git a/src/utils/auth/utils.ts b/src/utils/auth/utils.ts
index fec3e572..b15b8455 100644
--- a/src/utils/auth/utils.ts
+++ b/src/utils/auth/utils.ts
@@ -28,13 +28,15 @@ export function authGitHub(
       show: true,
     });
 
-    const githubUrl = `https://${authOptions.hostname}/login/oauth/authorize`;
-    const authUrl = `${githubUrl}?client_id=${authOptions.clientId}&scope=${Constants.AUTH_SCOPE}`;
+    const authUrl = new URL(`https://${authOptions.hostname}`);
+    authUrl.pathname = '/login/oauth/authorize';
+    authUrl.searchParams.append('client_id', authOptions.clientId);
+    authUrl.searchParams.append('scope', Constants.AUTH_SCOPE.toString());
 
     const session = authWindow.webContents.session;
     session.clearStorageData();
 
-    authWindow.loadURL(authUrl);
+    authWindow.loadURL(authUrl.toString());
 
     const handleCallback = (url: Link) => {
       const raw_code = /code=([^&]*)/.exec(url) || null;


Test patch:
diff --git a/src/utils/auth/utils.test.ts b/src/utils/auth/utils.test.ts
index 9f839959..914e90de 100644
--- a/src/utils/auth/utils.test.ts
+++ b/src/utils/auth/utils.test.ts
@@ -47,7 +47,7 @@ describe('utils/auth/utils.ts', () => {
 
       expect(loadURLMock).toHaveBeenCalledTimes(1);
       expect(loadURLMock).toHaveBeenCalledWith(
-        'https://github.com/login/oauth/authorize?client_id=FAKE_CLIENT_ID_123&scope=read:user,notifications,repo',
+        'https://github.com/login/oauth/authorize?client_id=FAKE_CLIENT_ID_123&scope=read%3Auser%2Cnotifications%2Crepo',
       );
 
       expect(browserWindow.destroy).toHaveBeenCalledTimes(1);
Repository: gitify-app/gitify
Pull request #1313 (2024-06-25 15:12:32)

feat: increase tooltip width

bmulholland: Also, a nit: if it's a UI change, it's not a refactor :)

Patch:
diff --git a/src/components/fields/Tooltip.tsx b/src/components/fields/Tooltip.tsx
index fd83a9c1..bd3a2e05 100644
--- a/src/components/fields/Tooltip.tsx
+++ b/src/components/fields/Tooltip.tsx
@@ -19,7 +19,7 @@ export const Tooltip: FC<ITooltip> = (props: ITooltip) => {
     >
       <QuestionIcon className="ml-1 text-blue-500" />
       {showTooltip && (
-        <div className="absolute z-10 rounded border border-gray-300 bg-white p-2 shadow dark:bg-gray-sidebar">
+        <div className="absolute z-10 w-60 rounded border border-gray-300 bg-white p-2 shadow dark:bg-gray-sidebar">
           <div className="text-left text-xs text-gray-700 dark:text-white">
             {props.tooltip}
           </div>
diff --git a/src/routes/Settings.tsx b/src/routes/Settings.tsx
index fe76b6b6..fada5716 100644
--- a/src/routes/Settings.tsx
+++ b/src/routes/Settings.tsx
@@ -244,7 +244,7 @@ export const SettingsRoute: FC = () => {
             }
             tooltip={
               <div>
-                When enabled you can choose to use the hotkeys{' '}
+                When enabled you can use the hotkeys{' '}
                 <span className="text-orange-600">
                   {Constants.DEFAULT_KEYBOARD_SHORTCUT}
                 </span>{' '}


Test patch:
diff --git a/src/components/fields/__snapshots__/Tooltip.test.tsx.snap b/src/components/fields/__snapshots__/Tooltip.test.tsx.snap
index b650726a..e40637c4 100644
--- a/src/components/fields/__snapshots__/Tooltip.test.tsx.snap
+++ b/src/components/fields/__snapshots__/Tooltip.test.tsx.snap
@@ -21,7 +21,7 @@ exports[`components/fields/Tooltip.tsx should display on mouse enter / leave 1`]
     />
   </svg>
   <div
-    class="absolute z-10 rounded border border-gray-300 bg-white p-2 shadow dark:bg-gray-sidebar"
+    class="absolute z-10 w-60 rounded border border-gray-300 bg-white p-2 shadow dark:bg-gray-sidebar"
   >
     <div
       class="text-left text-xs text-gray-700 dark:text-white"
Repository: Hopding/pdf-lib
Pull request #429 (2020-04-29 00:00:59)

Handle non ASCII characters in PDF name objects


Patch:
diff --git a/src/core/parser/PDFObjectParser.ts b/src/core/parser/PDFObjectParser.ts
index 20e1ba4..ec3e8a7 100644
--- a/src/core/parser/PDFObjectParser.ts
+++ b/src/core/parser/PDFObjectParser.ts
@@ -140,14 +140,7 @@ class PDFObjectParser extends BaseParser {
     let name = '';
     while (!this.bytes.done()) {
       const byte = this.bytes.peek();
-      if (
-        byte < CharCodes.ExclamationPoint ||
-        byte > CharCodes.Tilde ||
-        IsWhitespace[byte] ||
-        IsDelimiter[byte]
-      ) {
-        break;
-      }
+      if (IsWhitespace[byte] || IsDelimiter[byte]) break;
       name += charFromCode(byte);
       this.bytes.next();
     }


Test patch:
diff --git a/tests/core/parser/PDFObjectParser.spec.ts b/tests/core/parser/PDFObjectParser.spec.ts
index 0cd4578..9bc117f 100644
--- a/tests/core/parser/PDFObjectParser.spec.ts
+++ b/tests/core/parser/PDFObjectParser.spec.ts
@@ -306,6 +306,10 @@ describe(`PDFObjectParser`, () => {
       expect(parser.parseObject()).toBe(PDFName.of('Bing'));
       expect(parser.parseObject()).toBe(PDFName.of('Bang'));
     });
+
+    it(`handles names containing non-ASCII characters`, () => {
+      expectParse('/ABCDEE+»ªÎÄÖÐËÎ').toBe(PDFName.of('ABCDEE+»ªÎÄÖÐËÎ'));
+    });
   });
 
   describe(`when parsing arrays`, () => {