fix: nonce rpc checks

.changeset/nervous-bees-whisper.md

@@ -0,0 +1,5 @@
+---
+'@eth-optimism/l2geth': patch
+---
+
+Return a better error message for when the nonce is too high. Previously it would return `nonce too low` for even when the nonce was too high. Now it will return `nonce too high` to the user

.changeset/yellow-yaks-brake.md

@@ -0,0 +1,5 @@
+---
+'@eth-optimism/integration-tests': patch
+---
+
+Add tests for nonce too low and too high

integration-tests/test/rpc.spec.ts

@@ -72,6 +72,32 @@ describe('Basic RPC tests', () => {
       expect(result.data).to.equal(tx.data)
     })
 
+    it('should reject a transaction with a too low of nonce', async () => {
+      const tx = {
+        ...defaultTransactionFactory(),
+        gasPrice: await gasPriceForL2(env),
+        nonce: (await wallet.getTransactionCount()) - 1,
+      }
+
+      const signed = await wallet.signTransaction(tx)
+      await expect(provider.sendTransaction(signed)).to.be.rejectedWith(
+        'invalid transaction: nonce too low'
+      )
+    })
+
+    it('should reject a transaction with a too high of a nonce', async () => {
+      const tx = {
+        ...defaultTransactionFactory(),
+        gasPrice: await gasPriceForL2(env),
+        nonce: (await wallet.getTransactionCount()) + 10,
+      }
+
+      const signed = await wallet.signTransaction(tx)
+      await expect(provider.sendTransaction(signed)).to.be.rejectedWith(
+        'invalid transaction: nonce too high'
+      )
+    })
+
     it('should not accept a transaction with the wrong chain ID', async () => {
       const tx = {
         ...defaultTransactionFactory(),

l2geth/core/tx_pool.go

@@ -555,8 +555,11 @@ func (pool *TxPool) validateTx(tx *types.Transaction, local bool) error {
 	}
 	// Ensure the transaction adheres to nonce ordering
 	if rcfg.UsingOVM {
-		if pool.currentState.GetNonce(from) != tx.Nonce() {
+		nonce := pool.currentState.GetNonce(from)
+		if nonce > tx.Nonce() {
 			return ErrNonceTooLow
+		} else if nonce < tx.Nonce() {
+			return errors.New("nonce too high")
 		}
 	} else {
 		if pool.currentState.GetNonce(from) > tx.Nonce() {