Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Remove unused ui-test folder #202

Merged
merged 1 commit into from
Aug 15, 2022
Merged

Conversation

acuarica
Copy link
Contributor

This PR removes the ui-test folder since it is not maintained anymore. Moreover, by removing it, we keep the codebase easier to understand.

All our tests belong into the test folder.

Copy link
Contributor

@xhulz xhulz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@michaeljohnbennett michaeljohnbennett merged commit ce75b2c into develop Aug 15, 2022
@michaeljohnbennett michaeljohnbennett deleted the fix/remove-ui-test branch August 15, 2022 09:43
michaeljohnbennett added a commit that referenced this pull request Aug 24, 2022
* feat: saving the progress

* feat: saving the progress

* feat: saving the progress

* Started the cleanup of eslint and pre-commit hooks.

Will enforce line style in next commit.

* Enforcing line indents to 2. 💥

* Adding some files to trigger a husky preHook.

* More husky config.

* Enforcing line indents to 2. Formatting test. 💥

* come on husky... you can do it.

* chore: linting

* chore: 🚨 linting.

* fixing the prettier/eslint configs and removing conflicts.

* chore: fixing all the quotes to single quote. Yeah it's a thing.

* chore: removing inferrable types.

* chore: fixing no-unused-vars. Made _vars ignored.

All other instances have been  fixed. 🩹

* chore: fixing namespace/module issue.

* chore: Removing some more rules we turned off.

* chore: Removing type rules that were disabling a few checks.

* feat: Dashboard box has been implemented

* feat: Tree view implementation has changed from AZTreeView to TreeView

* feat: The Dashboard Service has removed from Networks box

* test: The dashboard test suite has implemented

* fix: Code tweaks

* fix: Network view has moved to the top

* test: More tests have added to simulate deployment via dashboard

* fix: Dashboard processes (sometimes) was not being removed from output channel

* fix: Code for the refactoring of the truffle config readers.

* chore: adding in deleted file.

* fix: The mnemonic. nemoonik, mnemonoomic. What a word.

Random fact, Keanu Reeves starred as Johnny Mnemonic in the movie of the same name. 🤖 📝

His brain could hold 80Gigs of data. Pitiful. He should have used his iphone or iCloud instead. (There are other cloud options for storing your private data in...)

* fix: final changes to get tests working!

* fix: The issue from debug a project inside a subdirectory was fixed

* fix: Tests issues have fixed

* chore: Some functions name has changed. The truffleWorkspace has chenged to WorkspaceFolder. Put the option to Debug a transaction ocontract menu

* fix: rolled backed to TruffleWorkspace because workspace folder is not working with windows volume mapping

* fix: final bits of config.

* fix: adding commitlint

This checks the syntax of our commits. adding some constancy to the
 process of commiting with a decent message.

A rod for my own back.

* chore: fixing the docs

Adding more info in the CONTRIBUTING.md file for the correct syntax. 📝

* fix: Maximum version of apps was set on required apps page

* renamed from explorer - truffle to contract explorer

issue #121

* fix: webpack config changes

This will repair the webpack folder issues we were seeing on builds.

* chore: changes was made in order to follow the code review suggestions

* feat(explorer): added createContract

fix(commands): fixed typo "Plataform" => "Platform"

* fix(createContract): refresh explorer after createContract

* fix: updating changelog

bumping to 2.3.5 in package.json and testing local vsix.

* fix: added new PR checks

This adds in new CI tasks for PR checks.

Ensures our code quality/formatting standards are met.

:alembic:

* fix: added new PR checks/renamed on develop

Had to commit here as it won't run on a feature branch.

This adds in new CI tasks for PR checks.

Ensures our code quality/formatting standards are met.

:alembic:

Also renamed the main.yml to publish.yml and enabled into release action
.

* chore: fixing a few things

Prettier was upset about some changes. :rotating_light:

* chore: windowz and prettier fixes

eol/linefeeds. :construction_worker:

* chore: hacks reintroduced

The ts-patch code needs to be added in the beginning to allow us to
patch imports.

* fix: ci fixes

Problems with the way CI sends variables into the action.

Fixing the variable names.

* fix: hdwallet npm package has updated to truffle/hdwallet-provider

* fix: trying to get the paths correct

Pre-Release/Release fixes.

* fix: error with GitHub action

* fix: make action trigger-able

* fix: look at changelog code

Adding this action in to test changelog generation.

* chore: prettifying all the things

* fix: reworking CI Files

making reusable workflows.

Testing out the triggers for actions.

* fix: fixing paths

Disabling the changelog.yml file for the moment.

* fix: auto load has implemented

* refactor: remove unused code that checks whether Python is installed

* fix: start ganache command when invoked from the command palette (#142)

* fix: activationEvents list commands has reverted (#167)

* fix: activationEvents list commands has reverted

* fix: more events has added to the list

* fix: fork network is working with deploy (#146)

* Draft: Required app was validating NPM local packages (#164)

* fix: npm local version validation has removed from truffle and ganache

* fix: rollbacking the activationEvents list

* feat: required app page fix

Added in the action and link for the requirements page right now.

Stops a random error appearing otherwise. Found this while testing.

Co-authored-by: MB <951378+michaeljohnbennett@users.noreply.github.com>

* fix: enabling the action for changelog

Adding in the ability to manually trigger the changelog action so we can
 use it as part of the release process.

* chore: updating changelog

* chore: removing metadata

Somehow we generated metadata into the package.json file which mucks up
pre-release versions.

* fix: bumping released version

Adding in new Version to changelog diff v2.4.0

This will be the published version from master.

* fix: making preRelease true

Making the flag true by default. There is some kind of issue herex
I needed to fix to make pre-release builds work again.

Thanks VSCode for silently updating things and not telling me why.

* fix: add new tasks to publish

The CI task needs to force these flags to ensure the builds are going to
 be correctly configured for the marketplace.

* fix: adjust ignore file

* Fixes the Debug Transaction command (#174)

* fix: revert some changes from #119 to fix debugger

* fix: remove duplicated types

* Dashboard link has added to dashboard view panel (#172)

feat: dashboard link has added to dashboard view panel

* Debug only works with Ganache Development Instance (#170)

* fix: a network selector has implemented on deploy command

* fix: refactoring for removing extra code

* fix: dead code has been removed

* Clean up duplicated Command IDs (#178)

fix: clean up duplicated Command IDs

* Honor the `contract_build_directory` property in Deployments view (#179)

* fix: honor the `contract_build_directory` in deployments

* refactor: simplify Deployment and Help views

* docs: improve Help & Deployment TS docs

* chore: remove unused `contextValue`

* fix: move `Network Deployments` item as last child

* Ability to compile a single contract (#177)

* feat: ability to compile a single contract

* feat: the menu has been implemented according to the element option

* Ability to debug over a forked network (#186)

* chore: saving progress

* refactor: non essencial code has removed

* refactor: lodash has been removed

* Add support for multiple Truffle config files (#181)

* feat: add initial support for multiple truffle config files

* fix: enable create contract only when a workspace is open

* feat: enable support for multiple Truffle config files

* docs: improve ts docs of workspaces

* docs: improve ts doc in `workspace`

* fix: update Truffle config quick pick placeholder

* feat: support multiple Truffle config in Deployments

* refactor: use a union type for possible path directory keys

* docs: improve Deployments view TS docs

* feat: improve error handling in Deployments view

* Add support to run out tests without  VS Code Development Extension Host (#191)

* feat: add initial support to run tests outside vscode runner

* fix: adapt all tests to be able to run without VS Code Runner

* feat: add script to run mock tests

It might be even more confortable to just run `mocha` instead of `test:mock`.
However, the goal of the script is to be more declarative
about possible commands to run.

* docs: improve ts docs for mocha config

* chore: make `yarn test` default to `mocha`

* feat: run integration tests in CI

* docs: add integration test to Development section

* Fix Command Palette display filter for Build/Deploy/Create Contracts commands (#192)

* fix: use proper Command Palette filters for build/deploy/create

* test: add more cases for `getWorkspaceRoot`

* fix: restore `getTruffleWorkspace` original behavior

* Use `contracts_directory` from Truffle config in `Create Contract` command (#193)

* fix: create contract command honors `contracts_directory`

* docs: improve TS docs for `createContract`

* Replace `open` package with `vscode.env.openExternal` (#195)

* fix: replace `open` package with `vscode.env.openExternal`

* fix: update lock file after `open` package removal

* Remove `vscode-azext-utils` and refactor `OutputChannel` (#197)

* refactor: remove `vscode-azext-utils` usages

The package can be removed now. Moreover, this commit
refactors the OutputChannel usages.

* fix: remove `vscode-azext-utils` package

* Add a Walkthrough on the Getting Started splash (#148)

* feat: add extension walktrough

* feat: improve walkthrough step descriptions

* feat: add Resources and Explore Commands steps

* feat: add deploy contract step to Walkthrough

* fix: minor walkthrough wording tweaks

* fix: removing legacy welcome screen

* Revert "fix: removing legacy welcome screen"

This reverts commit 6e269d6.

* Revert "fix: minor walkthrough wording tweaks"

This reverts commit 09724ff.

* fix: replay Kevin's changes

* feat: replace outdated images/gifs

Co-authored-by: MB <951378+michaeljohnbennett@users.noreply.github.com>
Co-authored-by: Kevin Bluer <kevin@bluer.com>

* Remove Mocha config from packaged extension (#198)

chore: remove mocha config from packaged extension

* Remove unused `dom` `lib` from `tsconfig` (#199)

chore: remove unused `dom` `lib` from `tsconfig`

* Remove unused `ui-test` folder (#202)

chore: remove old/unused `ui-test` folder

* fix: update import path in order to be able ton run test integration (script test:int) (#206)

fix: update import path in order to be able ton run script test:int

* Add PR template

* build: add to changelog haiku release (#213)

Co-authored-by: xhulz <wpmwsc@gmail.com>
Co-authored-by: MB <951378+michaeljohnbennett@users.noreply.github.com>
Co-authored-by: Nikolai <94403162+NikolaiSch@users.noreply.github.com>
Co-authored-by: Marcos Schulz <32708942+xhulz@users.noreply.github.com>
Co-authored-by: Kevin Bluer <kevin@bluer.com>
Co-authored-by: Luc Murat <108140021+luc-murat@users.noreply.github.com>
Co-authored-by: Alexandra Tran <alexandratran@protonmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants