test: calculator widget + dev integration tests setup - #934

Closed
ovitrif wants to merge 5 commits into
masterfrom
test/calculator-widget-device-tests
Closed

test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom
test/calculator-widget-device-tests

Conversation

@ovitrif

@ovitrifovitrif commented May 7, 2026•
edited
Loading

Copy link
Copy Markdown
Collaborator

Fixes#937

Description

This PR:

  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.

Preview

N/A

QA Notes

Manual Tests

  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.

Automated Tests

  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest

@ovitrifovitrif self-assigned this May 7, 2026
@ovitrifovitrif added this to the 2.3.0 milestone May 7, 2026
@ovitrifovitrif mentioned this pull request May 7, 2026
Merged
@ovitrifovitrif linked an issue May 7, 2026 that may be closed by this pull request
Polish Calculator Widget Fiat Input #937
Closed
Base automatically changed from fix/currency-widget-consistency-881 to masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif changed the title test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif changed the title test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif mentioned this pull request May 15, 2026
Merged
42 tasks
@ovitrif

ovitrif commented May 17, 2026

Copy link
Copy Markdown
CollaboratorAuthor

Replaced by:

Reconstructed on top of refactored calculator widget introduced by:

@ovitrifovitrif closed this May 17, 2026
@ovitrif
ovitrif deleted the test/calculator-widget-device-tests branch May 17, 2026 23:29
@ovitrifovitrif removed a link to an issue May 20, 2026
Polish Calculator Widget Fiat Input #937
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ovitrifovitrif

Labels

None yet

Projects

None yet

Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
test: calculator widget + dev integration tests setup - #934
Closed
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Closed
test: calculator widget + dev integration tests setup#934
ovitrif wants to merge 5 commits into
masterfrom 
test/calculator-widget-device-tests
Conversation
@ovitrif
@ovitrifovitrif
commented
May 7, 2026•

edited

Loading

Copy link


Copy Markdown

Collaborator
Fixes#937
Description
This PR:
  1. Moves calculator widget input/conversion behavior into the calculator view model so the card only renders state and forwards events.
  2. Adds unit coverage for calculator hydration, sanitizing, conversion, persistence, and currency refresh behavior.
  3. Adds a device UI integration test for the calculator card that types into BTC and fiat inputs, verifies rendered values, and verifies persisted widget state.
  4. Splits Android instrumentation tests into annotated lanes for Compose UI, device UI integration, device storage integration, and core service integration.
Preview
N/A
QA Notes
Manual Tests
  • 1. Widgets → Calculator Preview → enter BTC amount: fiat field updates and saved widget keeps both values.
  • 2. Widgets → Calculator Preview → enter fiat amount: BTC field updates and saved widget keeps both values.
Automated Tests
  • Unit tests added: cover calculator view model behavior in CalculatorViewModelTest.kt.
  • Android test added: covers calculator card device interaction and persistence in CalculatorCardIntegrationTest.kt.
  • Android test lanes added:
  • connectedDevDebugComposeAndroidTest
  • connectedDevDebugDeviceUiIntegrationAndroidTest
  • connectedDevDebugDeviceStorageIntegrationAndroidTest
  • connectedDevDebugCoreServiceIntegrationAndroidTest
  • connectedDevDebugDeviceIntegrationAndroidTest
Checks
  • Ran locally: ./gradlew compileDevDebugKotlin.
  • Ran locally: ./gradlew compileDevDebugAndroidTestKotlin.
  • Ran locally: ./gradlew testDevDebugUnitTest.
  • Ran locally: ./gradlew detekt --rerun-tasks.
  • Ran on Pixel 10 Pro:
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceUiIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceStorageIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugCoreServiceIntegrationAndroidTest
ANDROID_SERIAL='$ID' ./gradlew connectedDevDebugDeviceIntegrationAndroidTest
@ovitrifovitrif
self-assigned this
May 7, 2026
@ovitrifovitrif
added this to the 2.3.0 milestone
May 7, 2026
@ovitrifovitrif
mentioned this pull request
May 7, 2026
 Merged

@ovitrifovitrif
linked an issue
May 7, 2026
that may be
closed
by this pull request


Polish Calculator Widget Fiat Input
#937

Closed


Base automatically changed from
fix/currency-widget-consistency-881 to
masterMay 7, 2026 16:36
@ovitrif
Merge branch 'master' into test/calculator-widget-device-tests
076b45b
# Conflicts:
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/blocks/BlocksPreviewScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsEditScreenTest.kt
#	app/src/androidTest/java/to/bitkit/ui/screens/widgets/facts/FactsPreviewScreenTest.kt
@ovitrifovitrif
changed the title
test: add calculator device coveragetest: calculator widgetMay 12, 2026
@ovitrifovitrif
changed the title
test: calculator widgettest: calculator widget + dev integration tests setupMay 12, 2026
@ovitrifovitrif
mentioned this pull request
May 15, 2026
 Merged

42 tasks
@ovitrif
ovitrif
commented
May 17, 2026

Copy link


Copy Markdown

CollaboratorAuthor
Replaced by:
Reconstructed on top of refactored calculator widget introduced by:
@ovitrifovitrif
closed this
May 17, 2026
@ovitrif
ovitrif
deleted the

test/calculator-widget-device-tests

branch
May 17, 2026 23:29
@ovitrifovitrif
removed a link to an issue
May 20, 2026

Polish Calculator Widget Fiat Input
#937

Closed

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

Reviewers


No reviews


Assignees

@ovitrifovitrif

Labels


None yet


Projects


None yet


Milestone

2.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@ovitrif