Skip to content

Commit

Permalink
Refactor message converters and messages in flutter_custom_tabs_ios
Browse files Browse the repository at this point in the history
  • Loading branch information
droibit committed Dec 6, 2023
1 parent 273d749 commit 4e72c24
Show file tree
Hide file tree
Showing 10 changed files with 10 additions and 14 deletions.
2 changes: 1 addition & 1 deletion flutter_custom_tabs_ios/example/lib/main.dart
Original file line number Diff line number Diff line change
Expand Up @@ -153,4 +153,4 @@ Future<void> _launchInExternalBrowser() async {
} catch (e) {
debugPrint(e.toString());
}
}
}
5 changes: 2 additions & 3 deletions flutter_custom_tabs_ios/lib/src/custom_tabs_plugin_ios.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import 'package:flutter_custom_tabs_platform_interface/flutter_custom_tabs_platform_interface.dart';

import 'message_converters.dart';
import 'messages.g.dart';
import 'types/safari_view_controller_options.dart';
import 'messages/messages.dart';
import 'types/types.dart';

/// The iOS implementation of [CustomTabsPlatform].
///
Expand Down
2 changes: 2 additions & 0 deletions flutter_custom_tabs_ios/lib/src/messages/messages.dart
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
export 'messages.g.dart';
export 'type_conversion.dart';
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import 'package:flutter/painting.dart';

import 'types/types.dart';
import 'messages.g.dart';
import '../types/types.dart';

extension SafariViewControllerOptionsConverter on SafariViewControllerOptions {
SafariViewControllerOptionsMessage toMessage() {
Expand Down
2 changes: 1 addition & 1 deletion flutter_custom_tabs_ios/pigeons/messages.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import 'package:pigeon/pigeon.dart';

@ConfigurePigeon(PigeonOptions(
swiftOut: 'ios/Classes/messages.g.swift',
dartOut: 'lib/src/messages.g.dart',
dartOut: 'lib/src/messages/messages.g.dart',
))
@HostApi()
abstract class CustomTabsApi {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import 'package:flutter_custom_tabs_ios/flutter_custom_tabs_ios.dart';
import 'package:flutter_custom_tabs_ios/src/messages/messages.dart';
import 'package:flutter_custom_tabs_platform_interface/flutter_custom_tabs_platform_interface.dart';
import 'package:flutter_test/flutter_test.dart';

import 'messages.dart';

void main() {
late _MockCustomTabsApi api;
late CustomTabsPluginIOS customTabs;
Expand Down
2 changes: 0 additions & 2 deletions flutter_custom_tabs_ios/test/messages.dart

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import 'package:flutter/painting.dart';
import 'package:flutter_custom_tabs_ios/flutter_custom_tabs_ios.dart';
import 'package:flutter_custom_tabs_ios/src/messages/messages.dart';
import 'package:flutter_test/flutter_test.dart';

import '../messages.dart';

void main() {
group('SafariViewControllerOptions', () {
test('toMessage() returns empty message when option values are null', () {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import 'package:flutter_custom_tabs_ios/flutter_custom_tabs_ios.dart';
import 'package:flutter_custom_tabs_ios/src/messages/messages.dart';
import 'package:flutter_test/flutter_test.dart';

import '../messages.dart';

void main() {
group('SheetPresentationControllerConfiguration', () {
test('toMessage() returns expected message with default values', () {
Expand Down

0 comments on commit 4e72c24

Please sign in to comment.