From db422fab7e38e769a164a67221634c9bfc77ee60 Mon Sep 17 00:00:00 2001
From: Uri Shaked
Date: Fri, 29 May 2020 11:23:49 +0300
Subject: fix(gpio): port state not updated on DDR write

Calling `pinState()` inside a GPIO port listener returns incorrect values after changing DDR

close #47
---
 src/peripherals/gpio.spec.ts | 13 +++++++++++++
 1 file changed, 13 insertions(+)

(limited to 'src/peripherals/gpio.spec.ts')

diff --git a/src/peripherals/gpio.spec.ts b/src/peripherals/gpio.spec.ts
index 1dfaf7d..44232f4 100644
--- a/src/peripherals/gpio.spec.ts
+++ b/src/peripherals/gpio.spec.ts
@@ -92,6 +92,19 @@ describe('GPIO', () => {
       cpu.writeData(0x25, 0x01); // PORTB <- 0x01
       expect(listener).toHaveBeenCalled();
     });
+
+    it('should reflect the current port state when called inside a listener after DDR change', () => {
+      // Related issue: https://github.com/wokwi/avr8js/issues/47
+      const cpu = new CPU(new Uint16Array(1024));
+      const port = new AVRIOPort(cpu, portBConfig);
+      const listener = jest.fn(() => {
+        expect(port.pinState(0)).toBe(PinState.Low);
+      });
+      expect(port.pinState(0)).toBe(PinState.Input);
+      port.addListener(listener);
+      cpu.writeData(0x24, 0x01); // DDRB <- 0x01
+      expect(listener).toHaveBeenCalled();
+    });
   });
 
   describe('setPin', () => {
-- 
cgit v1.2.3