Skip to content

Commit

Permalink
Deleting a bunch of code that is no longer used
Browse files Browse the repository at this point in the history
Everything should still build (and tests are green). CI will complain if
I missed something.

Fixes #2080
  • Loading branch information
jleyba committed May 19, 2016
1 parent c76adf8 commit 119021b
Show file tree
Hide file tree
Showing 60 changed files with 43 additions and 20,665 deletions.
7 changes: 0 additions & 7 deletions Rakefile
Original file line number Diff line number Diff line change
Expand Up @@ -433,13 +433,6 @@ end
desc "Calculate dependencies required for testing the automation atoms"
task :calcdeps => "build/javascript/deps.js"

task :test_webdriverjs => [
"//javascript/webdriver:es6_test:run"
]

desc "Generate a single file with WebDriverJS' public API"
task :webdriverjs => [ "//javascript/webdriver:webdriver" ]

desc "Repack jetty"
task "repack-jetty" => "build/third_party/java/jetty/jetty-repacked.jar"

Expand Down
2 changes: 0 additions & 2 deletions javascript/remote/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,7 @@ js_binary(name = 'client',
srcs = glob(['*.js', 'ui/*.js',]),
deps = [
'//javascript/atoms:errors',
'//javascript/webdriver:base',
'//javascript/webdriver:http',
'//javascript/webdriver:session',
'//third_party/closure:closure',
],
flags = [
Expand Down
4 changes: 2 additions & 2 deletions javascript/remote/build.desc
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ js_binary(name = "clientbin",
],
deps = [
"//javascript/atoms:all_js",
"//javascript/webdriver:webdriver_lib",
"//javascript/webdriver:http",
],
flags = [
"--jscomp_error=accessControls",
Expand Down Expand Up @@ -52,7 +52,7 @@ js_test(name = "test",
deps = [
"./**/*.js",
"//javascript/atoms:all_js",
"//javascript/webdriver:all_js",
"//javascript/webdriver:http",
"//java/client/test/org/openqa/selenium/javascript",
"//java/server/test/org/openqa/selenium:server-with-tests:uber",
])
8 changes: 4 additions & 4 deletions javascript/remote/ui/client.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ goog.provide('remote.ui.Client');

goog.require('bot.response');
goog.require('goog.Disposable');
goog.require('goog.Promise');
goog.require('goog.Uri');
goog.require('goog.array');
goog.require('goog.debug.Console');
Expand All @@ -33,7 +34,6 @@ goog.require('remote.ui.WebDriverScriptButton');
goog.require('webdriver.Command');
goog.require('webdriver.CommandName');
goog.require('webdriver.Session');
goog.require('webdriver.promise');



Expand Down Expand Up @@ -139,7 +139,7 @@ remote.ui.Client.prototype.disposeInternal = function() {
* Initializes the client and renders it into the DOM.
* @param {!Element=} opt_element The element to render to; defaults to the
* current document's BODY element.
* @return {!webdriver.promise.Promise} A promise that will be resolved when
* @return {!goog.Promise} A promise that will be resolved when
* the client has been initialized.
*/
remote.ui.Client.prototype.init = function(opt_element) {
Expand All @@ -166,7 +166,7 @@ remote.ui.Client.prototype.getSessionContainer = function() {
/**
* Executes a single command.
* @param {!webdriver.Command} command The command to execute.
* @return {!webdriver.promise.Promise} A promise that will be resolved with the
* @return {!goog.Promise} A promise that will be resolved with the
* command response.
* @private
*/
Expand All @@ -192,7 +192,7 @@ remote.ui.Client.prototype.logError_ = function(msg, e) {

/**
* Queries the server for its build info.
* @return {!webdriver.promise.Promise} A promise that will be resolved with the
* @return {!goog.Promise} A promise that will be resolved with the
* server build info.
* @private
*/
Expand Down
138 changes: 7 additions & 131 deletions javascript/webdriver/BUCK
Original file line number Diff line number Diff line change
@@ -1,51 +1,12 @@

js_library(name = 'base',
srcs = [
'command.js',
'events.js',
'promise.js',
'stacktrace.js',
],
deps = [
'//third_party/closure:closure',
],
visibility = [
'//javascript/remote:client',
'//javascript/safari-driver/...',
],
)

js_library(name = 'button',
srcs = [
'button.js',
],
)

js_library(name = 'capabilities',
srcs = [
'capabilities.js',
],
deps = [
':logging',
':serializable',
],
visibility = [
'//javascript/safari-driver/...'
],
)

js_library(name = 'events',
srcs = [
'events.js',
],
)

js_library(name = 'http',
srcs = glob(['http/*.js']),
srcs = glob([
'capabilities.js',
'command.js',
'http/*.js',
'logging.js',
'session.js',
]),
deps = [
':base',
':logging',
':promise',
'//javascript/atoms:errors',
'//third_party/closure:closure',
],
Expand All @@ -63,15 +24,6 @@ js_library(name = 'key',
],
)

js_library(name = 'locators',
srcs = [
'locators.js',
],
deps = [
'//third_party/closure:closure',
],
)

js_library(name = 'logging',
srcs = [
'logging.js',
Expand All @@ -84,51 +36,6 @@ js_library(name = 'logging',
],
)

js_library(name = 'promise',
srcs = [
'promise.js',
],
deps = [
':events',
':stacktrace',
'//third_party/closure:closure',
],
)

js_library(name = 'sequence',
srcs = [
'actionsequence.js',
'touchsequence.js',
],
deps = [
':base',
':button',
':key',
'//third_party/closure:closure',
],
)

js_library(name = 'serializable',
srcs = [
'serializable.js',
],
deps = [
],
)

js_library(name = 'session',
srcs = [
'session.js',
],
deps = [
':capabilities',
],
visibility = [
'//javascript/remote:client',
'//javascript/safari-driver/...'
],
)

js_library(name = 'stacktrace',
srcs = [
'stacktrace.js',
Expand All @@ -137,34 +44,3 @@ js_library(name = 'stacktrace',
'//third_party/closure:closure',
],
)

js_library(name = 'until',
srcs = [
'until.js',
],
deps = [
':locators',
'//javascript/atoms:errors',
'//third_party/closure:closure',
],
)

js_library(name = 'webdriver',
srcs = [
'webdriver.js',
],
deps = [
':base',
':key',
':locators',
':logging',
':sequence',
':session',
':until',
'//javascript/atoms:errors',
'//third_party/closure:closure',
],
visibility = [
'//javascript/safari-driver/...'
],
)
Loading

0 comments on commit 119021b

Please sign in to comment.