This repository has been archived by the owner on Jan 26, 2024. It is now read-only.
implement testLiquidationPaymentsOverbid #122
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dangerousfood
force-pushed
the
feat/liq-pay
branch
4 times, most recently
from
02fee46
to
84ea3cb
Compare
SantiagoGregory
previously requested changes
Nov 22, 2022
dangerousfood
force-pushed
the
feat/liq-pay
branch
from
84ea3cb
to
8489bda
Compare
androolloyd
added a commit
that referenced
this pull request
Nov 22, 2022
… squashed (#112) * seaport wip * removed AH * latest flow to account for insights from seaport team * fix for testCancelAuction to be correct * seaport wip * royalty engine added, with mock * add updates to loan proof gen, and seaport test dpeloyer * remove comments, disable mock for tests, needs to be fuzzed * remove dead code * more cleanup, mocks dynamic, use tokenID 99 for royalties enabled * cleanup, removing dead code, and gas golfing * golfing * natspec update * more natspec cleanup * mend more natspec cleanup * mend more natspec cleanup * mend more natspec cleanup * add liquidation fees calculator * updates for warping, cleanup in general * implemented collateraltoken errors * Fix install instructions and add markdown filetree (#116) * Fix install instructions and add markdown filetree * Cleaned up filetree * cleanup * tests passing using a combo of match and fufillment * tests passing - v3 changes * move match flow down so the buffer is closer to the actual bid * fix static values for the test * implement testNewLienExceeds2XEpoch() (#105) * implement testNewLienExceeds2XEpoch() * lien duration check * commitToLien now caps duration at 2 weeks * privatevault commitolienfix * add length cap on refinance * moved timeToSecondEpochEnd into method * minor changes * implement _timeToSecondEndIfPublic * remove isValidVault check in onlyVaults function Co-authored-by: = <santiagogregoryl@gmail.com> * replaced transferfrom with safertransferfrom (#111) * replaced transferfrom with safertransferfrom * update ierc721 Co-authored-by: Joseph Delong <joseph@delong.me> * implement testFinalAuctionEnd() (#115) * Refinance update (#110) * lien duration check * updates requested Co-authored-by: Joseph Delong <joseph@delong.me> * add sdk pointer for v3 fixes * Golfed CeilDiv🏌 * Hole 9 on LienToken * Finished up LienToken * Fixed syntax * clearing house is now a beacon * split opensea fees apart from seaport file * remove jsons for deploying and use compiled considerations * rather than looping the array copy it to memory then update the position with the new lien * removed log * Golfed ceilDiv and LienToken (#117) * Fix install instructions and add markdown filetree (#116) * Fix install instructions and add markdown filetree * Cleaned up filetree * implement testNewLienExceeds2XEpoch() (#105) * implement testNewLienExceeds2XEpoch() * lien duration check * commitToLien now caps duration at 2 weeks * privatevault commitolienfix * add length cap on refinance * moved timeToSecondEpochEnd into method * minor changes * implement _timeToSecondEndIfPublic * remove isValidVault check in onlyVaults function Co-authored-by: = <santiagogregoryl@gmail.com> * replaced transferfrom with safertransferfrom (#111) * replaced transferfrom with safertransferfrom * update ierc721 Co-authored-by: Joseph Delong <joseph@delong.me> * implement testFinalAuctionEnd() (#115) * Refinance update (#110) * lien duration check * updates requested Co-authored-by: Joseph Delong <joseph@delong.me> * Golfed CeilDiv🏌 * Hole 9 on LienToken * Finished up LienToken * Fixed syntax Co-authored-by: Joseph Delong <joseph@delong.me> Co-authored-by: = <santiagogregoryl@gmail.com> Co-authored-by: Andrew Redden <=> * remove reserve from the code, its unused, add validate stack guard on getMaxPotentialDebt * fixes and tests for adding liens against invalid stacks * fix invalid rename * getter to get opensea fee fees and payee data * fix typo * updates for uni v3 liquidiator to check the underlying token amounts if mins are set * cleanup and poitner update * liquidationInitialAsk Issue #99 (#121) * Added reverts for invalid liquidationInitialAsks * Added testing for liquidationInitialAsk revert cases * fixes and tests for adding liens against invalid stacks * fix invalid rename * getter to get opensea fee fees and payee data * fix typo * Fixed expectReverts for liquidationInitialAsk cases Co-authored-by: Andrew Redden <=> * Feat/exceeds initial ask (#120) * initial incorrect implementation * maybe broken liquidationinitialask test * fix maxpotentialdebt and added invalid hash enum * fix liquidationinitialask * implemented expectreverts in reverttesting * update testhelpers Co-authored-by: Andrew Redden <=> Co-authored-by: Andrew Redden <opensource@andrewredden.com> * implement testLiquidationPaymentsOverbid and testLiquidationNftTransfer (#122) * cleanup dead code * cleanup * remove unused check * remove validator asset * cleanup and more fixes * remove unused interface * remove comments * remove dead test * update readme * removed comments from test helper * more nits Co-authored-by: Andrew Redden <=> Co-authored-by: = <santiagogregoryl@gmail.com> Co-authored-by: Joseph Delong <joseph@delong.me> Co-authored-by: GregTheDev <40359730+0xgregthedev@users.noreply.github.com> Co-authored-by: GregTheDev <mapler98@gmail.com>
dangerousfood
added a commit
that referenced
this pull request
Nov 22, 2022
… squashed (#112) * seaport wip * removed AH * latest flow to account for insights from seaport team * fix for testCancelAuction to be correct * seaport wip * royalty engine added, with mock * add updates to loan proof gen, and seaport test dpeloyer * remove comments, disable mock for tests, needs to be fuzzed * remove dead code * more cleanup, mocks dynamic, use tokenID 99 for royalties enabled * cleanup, removing dead code, and gas golfing * golfing * natspec update * more natspec cleanup * mend more natspec cleanup * mend more natspec cleanup * mend more natspec cleanup * add liquidation fees calculator * updates for warping, cleanup in general * implemented collateraltoken errors * Fix install instructions and add markdown filetree (#116) * Fix install instructions and add markdown filetree * Cleaned up filetree * cleanup * tests passing using a combo of match and fufillment * tests passing - v3 changes * move match flow down so the buffer is closer to the actual bid * fix static values for the test * implement testNewLienExceeds2XEpoch() (#105) * implement testNewLienExceeds2XEpoch() * lien duration check * commitToLien now caps duration at 2 weeks * privatevault commitolienfix * add length cap on refinance * moved timeToSecondEpochEnd into method * minor changes * implement _timeToSecondEndIfPublic * remove isValidVault check in onlyVaults function Co-authored-by: = <santiagogregoryl@gmail.com> * replaced transferfrom with safertransferfrom (#111) * replaced transferfrom with safertransferfrom * update ierc721 Co-authored-by: Joseph Delong <joseph@delong.me> * implement testFinalAuctionEnd() (#115) * Refinance update (#110) * lien duration check * updates requested Co-authored-by: Joseph Delong <joseph@delong.me> * add sdk pointer for v3 fixes * Golfed CeilDiv🏌 * Hole 9 on LienToken * Finished up LienToken * Fixed syntax * clearing house is now a beacon * split opensea fees apart from seaport file * remove jsons for deploying and use compiled considerations * rather than looping the array copy it to memory then update the position with the new lien * removed log * Golfed ceilDiv and LienToken (#117) * Fix install instructions and add markdown filetree (#116) * Fix install instructions and add markdown filetree * Cleaned up filetree * implement testNewLienExceeds2XEpoch() (#105) * implement testNewLienExceeds2XEpoch() * lien duration check * commitToLien now caps duration at 2 weeks * privatevault commitolienfix * add length cap on refinance * moved timeToSecondEpochEnd into method * minor changes * implement _timeToSecondEndIfPublic * remove isValidVault check in onlyVaults function Co-authored-by: = <santiagogregoryl@gmail.com> * replaced transferfrom with safertransferfrom (#111) * replaced transferfrom with safertransferfrom * update ierc721 Co-authored-by: Joseph Delong <joseph@delong.me> * implement testFinalAuctionEnd() (#115) * Refinance update (#110) * lien duration check * updates requested Co-authored-by: Joseph Delong <joseph@delong.me> * Golfed CeilDiv🏌 * Hole 9 on LienToken * Finished up LienToken * Fixed syntax Co-authored-by: Joseph Delong <joseph@delong.me> Co-authored-by: = <santiagogregoryl@gmail.com> Co-authored-by: Andrew Redden <=> * remove reserve from the code, its unused, add validate stack guard on getMaxPotentialDebt * fixes and tests for adding liens against invalid stacks * fix invalid rename * getter to get opensea fee fees and payee data * fix typo * updates for uni v3 liquidiator to check the underlying token amounts if mins are set * cleanup and poitner update * liquidationInitialAsk Issue #99 (#121) * Added reverts for invalid liquidationInitialAsks * Added testing for liquidationInitialAsk revert cases * fixes and tests for adding liens against invalid stacks * fix invalid rename * getter to get opensea fee fees and payee data * fix typo * Fixed expectReverts for liquidationInitialAsk cases Co-authored-by: Andrew Redden <=> * Feat/exceeds initial ask (#120) * initial incorrect implementation * maybe broken liquidationinitialask test * fix maxpotentialdebt and added invalid hash enum * fix liquidationinitialask * implemented expectreverts in reverttesting * update testhelpers Co-authored-by: Andrew Redden <=> Co-authored-by: Andrew Redden <opensource@andrewredden.com> * implement testLiquidationPaymentsOverbid and testLiquidationNftTransfer (#122) * cleanup dead code * cleanup * remove unused check * remove validator asset * cleanup and more fixes * remove unused interface * remove comments * remove dead test * update readme * removed comments from test helper * more nits Co-authored-by: Andrew Redden <=> Co-authored-by: = <santiagogregoryl@gmail.com> Co-authored-by: Joseph Delong <joseph@delong.me> Co-authored-by: GregTheDev <40359730+0xgregthedev@users.noreply.github.com> Co-authored-by: GregTheDev <mapler98@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.