Skip to content

Commit

Permalink
Revert "[fuchsia] Adds a test for clock change (flutter#19993)" (flut…
Browse files Browse the repository at this point in the history
…ter#20790)

This reverts commit 3fe49ec.
  • Loading branch information
iskakaushik authored Aug 26, 2020
1 parent 619db58 commit 5952760
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 234 deletions.
1 change: 0 additions & 1 deletion ci/licenses_golden/licenses_flutter
Original file line number Diff line number Diff line change
Expand Up @@ -624,7 +624,6 @@ FILE: ../../../flutter/shell/common/serialization_callbacks.h
FILE: ../../../flutter/shell/common/shell.cc
FILE: ../../../flutter/shell/common/shell.h
FILE: ../../../flutter/shell/common/shell_benchmarks.cc
FILE: ../../../flutter/shell/common/shell_fuchsia_unittests.cc
FILE: ../../../flutter/shell/common/shell_io_manager.cc
FILE: ../../../flutter/shell/common/shell_io_manager.h
FILE: ../../../flutter/shell/common/shell_test.cc
Expand Down
11 changes: 0 additions & 11 deletions shell/common/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -263,16 +263,5 @@ if (enable_unittests) {
"//flutter/shell/version",
"//third_party/googletest:gmock",
]

if (is_fuchsia) {
sources += [ "shell_fuchsia_unittests.cc" ]

deps += [
"$fuchsia_sdk_root/fidl:fuchsia.intl",
"$fuchsia_sdk_root/fidl:fuchsia.settings",
"$fuchsia_sdk_root/pkg:fidl_cpp",
"$fuchsia_sdk_root/pkg:sys_cpp",
]
}
}
}
23 changes: 3 additions & 20 deletions shell/common/fixtures/shell_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -141,29 +141,12 @@ List<int> getFixtureImage() native 'GetFixtureImage';

void notifyLocalTime(String string) native 'NotifyLocalTime';

void waitFixture() native 'WaitFixture';

// Return local date-time as a string, to an hour resolution. So, "2020-07-23
// 14:03:22" will become "2020-07-23 14".
String localTimeAsString() {
@pragma('vm:entry-point')
void localtimesMatch() {
final now = DateTime.now().toLocal();
// This is: "$y-$m-$d $h:$min:$sec.$ms$us";
final timeStr = now.toString();
// Forward only "$y-$m-$d $h" for timestamp comparison. Not using DateTime
// formatting since package:intl is not available.
return timeStr.split(":")[0];
}

@pragma('vm:entry-point')
void localtimesMatch() {
notifyLocalTime(localTimeAsString());
}

@pragma('vm:entry-point')
void timezonesChange() {
notifyLocalTime(localTimeAsString());
waitFixture();
notifyLocalTime(localTimeAsString());
waitFixture();
notifyLocalTime(localTimeAsString());
notifyLocalTime(timeStr.split(":")[0]);
}
198 changes: 0 additions & 198 deletions shell/common/shell_fuchsia_unittests.cc

This file was deleted.

7 changes: 3 additions & 4 deletions testing/fuchsia/meta/fuchsia_test.cmx
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,13 @@
],
"services": [
"fuchsia.accessibility.semantics.SemanticsManager",
"fuchsia.process.Launcher",
"fuchsia.deprecatedtimezone.Timezone",
"fuchsia.logger.LogSink",
"fuchsia.netstack.Netstack",
"fuchsia.process.Launcher",
"fuchsia.settings.Intl",
"fuchsia.sysmem.Allocator",
"fuchsia.tracing.provider.Registry",
"fuchsia.vulkan.loader.Loader",
"fuchsia.logger.LogSink",
"fuchsia.tracing.provider.Registry",
"fuchsia.intl.PropertyProvider"
]
}
Expand Down

0 comments on commit 5952760

Please sign in to comment.