user twitter service to retrieve timelines

src/BirdsiteLive.Pipeline/Processors/RetrieveTweetsProcessor.cs

@@ -14,13 +14,13 @@ namespace BirdsiteLive.Pipeline.Processors
 {
     public class RetrieveTweetsProcessor : IRetrieveTweetsProcessor
     {
-        private readonly ITwitterService _twitterService;
+        private readonly ITwitterTweetsService _twitterTweetsService;
         private readonly ITwitterUserDal _twitterUserDal;
 
         #region Ctor
-        public RetrieveTweetsProcessor(ITwitterService twitterService, ITwitterUserDal twitterUserDal)
+        public RetrieveTweetsProcessor(ITwitterTweetsService twitterTweetsService, ITwitterUserDal twitterUserDal)
         {
-            _twitterService = twitterService;
+            _twitterTweetsService = twitterTweetsService;
             _twitterUserDal = twitterUserDal;
         }
         #endregion
@@ -56,9 +56,9 @@ namespace BirdsiteLive.Pipeline.Processors
         {
             ExtractedTweet[] tweets;
             if (user.LastTweetPostedId == -1)
-                tweets = _twitterService.GetTimeline(user.Acct, 1);
+                tweets = _twitterTweetsService.GetTimeline(user.Acct, 1);
             else
-                tweets = _twitterService.GetTimeline(user.Acct, 200, user.LastTweetSynchronizedForAllFollowersId);
+                tweets = _twitterTweetsService.GetTimeline(user.Acct, 200, user.LastTweetSynchronizedForAllFollowersId);
 
             return tweets;
         }

src/BirdsiteLive.Twitter/CachedTwitterService.cs

@@ -4,9 +4,9 @@ using Microsoft.Extensions.Caching.Memory;
 
 namespace BirdsiteLive.Twitter
 {
-    public class CachedTwitterService : ITwitterService
+    public class CachedTwitterUserService : ITwitterUserService
     {
-        private readonly ITwitterService _twitterService;
+        private readonly ITwitterUserService _twitterService;
 
         private MemoryCache _userCache = new MemoryCache(new MemoryCacheOptions()
         {
@@ -22,7 +22,7 @@ namespace BirdsiteLive.Twitter
             .SetAbsoluteExpiration(TimeSpan.FromDays(30));
 
         #region Ctor
-        public CachedTwitterService(ITwitterService twitterService)
+        public CachedTwitterUserService(ITwitterUserService twitterService)
         {
             _twitterService = twitterService;
         }
@@ -38,15 +38,5 @@ namespace BirdsiteLive.Twitter
 
             return user;
         }
-
-        public ExtractedTweet GetTweet(long statusId)
-        {
-            return _twitterService.GetTweet(statusId);
-        }
-
-        public ExtractedTweet[] GetTimeline(string username, int nberTweets, long fromTweetId = -1)
-        {
-            return _twitterService.GetTimeline(username, nberTweets, fromTweetId);
-        }
     }
 }

src/BirdsiteLive.Twitter/Models/TwitterUser.cs

@@ -2,6 +2,7 @@
 {
     public class TwitterUser
     {
+        public long Id { get; set; }
         public string Name { get; set; }
         public string Description { get; set; }
         public string Url { get; set; }

src/BirdsiteLive.Twitter/TwitterTweetsService.cs

@@ -1,65 +1,39 @@
-using System;
+using System.Collections.Generic;
-using System.Collections.Generic;
-using System.IO;
 using System.Linq;
-using System.Threading.Tasks;
 using BirdsiteLive.Common.Settings;
 using BirdsiteLive.Statistics.Domain;
 using BirdsiteLive.Twitter.Extractors;
 using BirdsiteLive.Twitter.Models;
 using Tweetinvi;
 using Tweetinvi.Models;
-using Tweetinvi.Models.Entities;
 using Tweetinvi.Parameters;
 
 namespace BirdsiteLive.Twitter
 {
-    public interface ITwitterService
+    public interface ITwitterTweetsService
     {
-        TwitterUser GetUser(string username);
         ExtractedTweet GetTweet(long statusId);
         ExtractedTweet[] GetTimeline(string username, int nberTweets, long fromTweetId = -1);
     }
 
-    public class TwitterService : ITwitterService
+    public class TwitterTweetsService : ITwitterTweetsService
     {
         private readonly TwitterSettings _settings;
         private readonly ITweetExtractor _tweetExtractor;
         private readonly ITwitterStatisticsHandler _statisticsHandler;
+        private readonly ITwitterUserService _twitterUserService;
 
         #region Ctor
-        public TwitterService(TwitterSettings settings, ITweetExtractor tweetExtractor, ITwitterStatisticsHandler statisticsHandler)
+        public TwitterTweetsService(TwitterSettings settings, ITweetExtractor tweetExtractor, ITwitterStatisticsHandler statisticsHandler, ITwitterUserService twitterUserService)
         {
             _settings = settings;
             _tweetExtractor = tweetExtractor;
             _statisticsHandler = statisticsHandler;
+            _twitterUserService = twitterUserService;
             Auth.SetApplicationOnlyCredentials(_settings.ConsumerKey, _settings.ConsumerSecret, true);
         }
         #endregion
-
+        
-        public TwitterUser GetUser(string username)
-        {
-            var user = User.GetUserFromScreenName(username);
-            _statisticsHandler.CalledUserApi();
-            if (user == null) return null;
-
-            // Expand URLs
-            var description = user.Description;
-            foreach (var descriptionUrl in user.Entities?.Description?.Urls?.OrderByDescending(x => x.URL.Length))
-                description = description.Replace(descriptionUrl.URL, descriptionUrl.ExpandedURL);
-
-            return new TwitterUser
-            {
-                Acct = username,
-                Name = user.Name,
-                Description = description,
-                Url = $"https://twitter.com/{username}",
-                ProfileImageUrl = user.ProfileImageUrlFullSize,
-                ProfileBackgroundImageUrl = user.ProfileBackgroundImageUrlHttps,
-                ProfileBannerURL = user.ProfileBannerURL
-            };
-        }
-
         public ExtractedTweet GetTweet(long statusId)
         {
             TweetinviConfig.CurrentThreadSettings.TweetMode = TweetMode.Extended;
@@ -72,9 +46,9 @@ namespace BirdsiteLive.Twitter
         public ExtractedTweet[] GetTimeline(string username, int nberTweets, long fromTweetId = -1)
         {
             TweetinviConfig.CurrentThreadSettings.TweetMode = TweetMode.Extended;
+            
+            var user = _twitterUserService.GetUser(username);
 
-            var user = User.GetUserFromScreenName(username);
-            _statisticsHandler.CalledUserApi();
             var tweets = new List<ITweet>();
             if (fromTweetId == -1)
             {
@@ -97,4 +71,4 @@ namespace BirdsiteLive.Twitter
             return tweets.Select(_tweetExtractor.Extract).ToArray();
         }
     }
-}
+}

src/BirdsiteLive.Twitter/TwitterUserService.cs

@@ -0,0 +1,55 @@
+using System.Linq;
+using BirdsiteLive.Common.Settings;
+using BirdsiteLive.Statistics.Domain;
+using BirdsiteLive.Twitter.Extractors;
+using BirdsiteLive.Twitter.Models;
+using Tweetinvi;
+
+namespace BirdsiteLive.Twitter
+{
+    public interface ITwitterUserService
+    {
+        TwitterUser GetUser(string username);
+    }
+
+    public class TwitterUserService : ITwitterUserService
+    {
+        private readonly TwitterSettings _settings;
+        private readonly ITweetExtractor _tweetExtractor;
+        private readonly ITwitterStatisticsHandler _statisticsHandler;
+
+        #region Ctor
+        public TwitterUserService(TwitterSettings settings, ITweetExtractor tweetExtractor, ITwitterStatisticsHandler statisticsHandler)
+        {
+            _settings = settings;
+            _tweetExtractor = tweetExtractor;
+            _statisticsHandler = statisticsHandler;
+            Auth.SetApplicationOnlyCredentials(_settings.ConsumerKey, _settings.ConsumerSecret, true);
+        }
+        #endregion
+
+        public TwitterUser GetUser(string username)
+        {
+            var user = User.GetUserFromScreenName(username);
+            _statisticsHandler.CalledUserApi();
+            if (user == null) return null;
+
+            // Expand URLs
+            var description = user.Description;
+            foreach (var descriptionUrl in user.Entities?.Description?.Urls?.OrderByDescending(x => x.URL.Length))
+                description = description.Replace(descriptionUrl.URL, descriptionUrl.ExpandedURL);
+
+            return new TwitterUser
+            {
+                Id = user.Id,
+                Acct = username,
+                Name = user.Name,
+                Description = description,
+                Url = $"https://twitter.com/{username}",
+                ProfileImageUrl = user.ProfileImageUrlFullSize,
+                ProfileBackgroundImageUrl = user.ProfileBackgroundImageUrlHttps,
+                ProfileBannerURL = user.ProfileBannerURL
+            };
+        }
+    }
+}

src/BirdsiteLive/Controllers/UsersController.cs

@@ -21,18 +21,20 @@ namespace BirdsiteLive.Controllers
 {
     public class UsersController : Controller
     {
-        private readonly ITwitterService _twitterService;
+        private readonly ITwitterUserService _twitterUserService;
+        private readonly ITwitterTweetsService _twitterTweetService;
         private readonly IUserService _userService;
         private readonly IStatusService _statusService;
         private readonly InstanceSettings _instanceSettings;
 
         #region Ctor
-        public UsersController(ITwitterService twitterService, IUserService userService, IStatusService statusService, InstanceSettings instanceSettings)
+        public UsersController(ITwitterUserService twitterUserService, IUserService userService, IStatusService statusService, InstanceSettings instanceSettings, ITwitterTweetsService twitterTweetService)
         {
-            _twitterService = twitterService;
+            _twitterUserService = twitterUserService;
             _userService = userService;
             _statusService = statusService;
             _instanceSettings = instanceSettings;
+            _twitterTweetService = twitterTweetService;
         }
         #endregion
 
@@ -53,7 +55,7 @@ namespace BirdsiteLive.Controllers
         public IActionResult Index(string id)
         {
             id = id.Trim(new[] { ' ', '@' }).ToLowerInvariant();
-            var user = _twitterService.GetUser(id);
+            var user = _twitterUserService.GetUser(id);
 
             var acceptHeaders = Request.Headers["Accept"];
             if (acceptHeaders.Any())
@@ -96,7 +98,7 @@ namespace BirdsiteLive.Controllers
                     if (!long.TryParse(statusId, out var parsedStatusId))
                         return NotFound();
 
-                    var tweet = _twitterService.GetTweet(parsedStatusId);
+                    var tweet = _twitterTweetService.GetTweet(parsedStatusId);
                     if (tweet == null)
                         return NotFound();
 

src/BirdsiteLive/Controllers/WellKnownController.cs

@@ -16,14 +16,14 @@ namespace BirdsiteLive.Controllers
     [ApiController]
     public class WellKnownController : ControllerBase
     {
-        private readonly ITwitterService _twitterService;
+        private readonly ITwitterUserService _twitterUserService;
         private readonly ITwitterUserDal _twitterUserDal;
         private readonly InstanceSettings _settings;
 
         #region Ctor
-        public WellKnownController(InstanceSettings settings, ITwitterService twitterService, ITwitterUserDal twitterUserDal)
+        public WellKnownController(InstanceSettings settings, ITwitterUserService twitterUserService, ITwitterUserDal twitterUserDal)
         {
-            _twitterService = twitterService;
+            _twitterUserService = twitterUserService;
             _twitterUserDal = twitterUserDal;
             _settings = settings;
         }
@@ -163,7 +163,7 @@ namespace BirdsiteLive.Controllers
             if (!string.IsNullOrWhiteSpace(domain) && domain != _settings.Domain)
                 return NotFound();
 
-            var user = _twitterService.GetUser(name);
+            var user = _twitterUserService.GetUser(name);
             if (user == null)
                 return NotFound();
 

src/BirdsiteLive/Startup.cs

@@ -84,8 +84,8 @@ namespace BirdsiteLive
                 throw new NotImplementedException($"{dbSettings.Type} is not supported");
             }
             
-            services.For<ITwitterService>().DecorateAllWith<CachedTwitterService>();
+            services.For<ITwitterUserService>().DecorateAllWith<CachedTwitterUserService>();
-            services.For<ITwitterService>().Use<TwitterService>().Singleton();
+            services.For<ITwitterUserService>().Use<TwitterUserService>().Singleton();
 
             services.Scan(_ =>
             {

src/Tests/BirdsiteLive.Pipeline.Tests/Processors/RetrieveTweetsProcessorTests.cs

@@ -40,7 +40,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors
             #endregion
 
             #region Mocks
-            var twitterServiceMock = new Mock<ITwitterService>(MockBehavior.Strict);
+            var twitterServiceMock = new Mock<ITwitterTweetsService>(MockBehavior.Strict);
             twitterServiceMock
                 .Setup(x => x.GetTimeline(
                     It.Is<string>(y => y == user1.Acct),
@@ -105,7 +105,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors
             #endregion
 
             #region Mocks
-            var twitterServiceMock = new Mock<ITwitterService>(MockBehavior.Strict);
+            var twitterServiceMock = new Mock<ITwitterTweetsService>(MockBehavior.Strict);
             twitterServiceMock
                 .Setup(x => x.GetTimeline(
                     It.Is<string>(y => y == user1.Acct),
@@ -165,7 +165,7 @@ namespace BirdsiteLive.Pipeline.Tests.Processors
             #endregion
 
             #region Mocks
-            var twitterServiceMock = new Mock<ITwitterService>(MockBehavior.Strict);
+            var twitterServiceMock = new Mock<ITwitterTweetsService>(MockBehavior.Strict);
             twitterServiceMock
                 .Setup(x => x.GetTimeline(
                     It.Is<string>(y => y == user1.Acct),