343 remove test stubs

[barakman]

Handles issue #343

[mikewcasale]

Why keep AM_SINGLE at all at this point? In bot.py I'd suggest removing the only usage as follows, and then delete the AM_SINGLE attribute definition at class level:

def _find_arbitrage(
        self,
        flashloan_tokens: List[str],
        CCm: CPCContainer,
        arb_mode: str = 'single',
        randomizer=int
    ) -> dict:
        random_mode = self.AO_CANDIDATES if randomizer else None
        arb_finder = self._get_arb_finder(arb_mode)
        finder = arb_finder(
            flashloan_tokens=flashloan_tokens,
            CCm=CCm,
            mode="bothin",
            result=random_mode,
            ConfigObj=self.ConfigObj,
        )
        return {"finder": finder, "r": finder.find_arbitrage()}

[barakman]

Why keep AM_SINGLE at all at this point? In bot.py I'd suggest removing the only usage as follows, and then delete the AM_SINGLE attribute definition at class level:

def _find_arbitrage(
        self,
        flashloan_tokens: List[str],
        CCm: CPCContainer,
        arb_mode: str = 'single',
        randomizer=int
    ) -> dict:
        random_mode = self.AO_CANDIDATES if randomizer else None
        arb_finder = self._get_arb_finder(arb_mode)
        finder = arb_finder(
            flashloan_tokens=flashloan_tokens,
            CCm=CCm,
            mode="bothin",
            result=random_mode,
            ConfigObj=self.ConfigObj,
        )
        return {"finder": finder, "r": finder.find_arbitrage()}

Thanks, that was my thought as well, but I noticed that AM_SINGLE was still used in various other places (primarily in test files), so I thought I'd keep the amount of changes in this PR to a minimum.
But now that you mention it, I'll have another look to see if I can clear that up as well.

[mikewcasale]

Looks good - except there are some unresolved conflicts with current main. Can you please resolve the conflicts so that the PR has no conflicts with main... then I'll approve.

[mikewcasale]

I'm not convinced the pro's outweigh the con's with all these changes. Convince me otherwise - and also address the specific comments concerning rawTransaction usage in multiple places.